Skip to content
This repository was archived by the owner on Mar 28, 2020. It is now read-only.

Commit dcbc34e

Browse files
committed
Fix comment about Intr*Mem properties
Summary: Follow up to D19291: it now makes sense to use two Intr*Mem properties, in particular IntrReadMem + IntrArgMemOnly is common. Pointed out by Mikael Holmén. Reviewers: uabelho, joker.eph, reames Subscribers: llvm-commits Differential Revision: http://reviews.llvm.org/D19418 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@267238 91177308-0d34-0410-b5e6-96231b3b80d8
1 parent 8a1d408 commit dcbc34e

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

include/llvm/IR/Intrinsics.td

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,7 @@ include "llvm/CodeGen/ValueTypes.td"
1919

2020
class IntrinsicProperty;
2121

22-
// Intr*Mem - Memory properties. An intrinsic is allowed to have at most one of
23-
// these properties set. They are listed from the most aggressive (best to use
24-
// if correct) to the least aggressive. If no property is set, the worst case
22+
// Intr*Mem - Memory properties. If no property is set, the worst case
2523
// is assumed (it may read and write any memory it can get access to and it may
2624
// have other side effects).
2725

0 commit comments

Comments
 (0)