[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