[PATCH] D50730: [AST] Generalize argument specific aliasing

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 7 10:33:06 PDT 2018


reames added inline comments.


================
Comment at: lib/Analysis/AliasSetTracker.cpp:457
+  // Handle all calls with known mod/ref sets generically, but special case the
+  // two family of intrinsics which can involve volatile access.
+  CallSite CS(I);
----------------
anna wrote:
> which two family of intrinsics? Are these the memset and memtransfer calls? They are handled above. Maybe move the comment to above line 451.
The volatile comment is stale and will be removed before landing.  AST no longer has a notion of volatile at all.


https://reviews.llvm.org/D50730





More information about the llvm-commits mailing list