[PATCH] D29695: FunctionAttrs: Factor out a function for querying memory access of a specific copy of a function. NFC.

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 10 20:04:03 PST 2017


tejohnson added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/FunctionAttrs.cpp:55
   FunctionModRefBehavior MRB = AAR.getModRefBehavior(&F);
   if (MRB == FMRB_DoesNotAccessMemory)
     // Already perfect!
----------------
Why don't we want to check this in the callers of computeFunctionBodyMemoryAccess? Isn't it a strong guarantee?


https://reviews.llvm.org/D29695





More information about the llvm-commits mailing list