[PATCH] D29989: [BasicAA] Take attributes into account when requesting modref info for a call site

Igor Laevsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 20 10:20:29 PST 2017


igor-laevsky added inline comments.


================
Comment at: lib/Analysis/BasicAliasAnalysis.cpp:786
+      if (CS.onlyReadsMemory(OperandNo)) {
+        Result = static_cast<ModRefInfo>(Result | MRI_Ref);
+        continue;
----------------
dberlin wrote:
> Can you explain why you need static casts here and below?
> 
This is to be in sync with similar places in this file. I don't really have a preference on how to convert this values.


https://reviews.llvm.org/D29989





More information about the llvm-commits mailing list