[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