[PATCH] D50679: [BasicAA] Don't assume tail calls with byval don't alias allocas
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 13 17:21:39 PDT 2018
hfinkel added a comment.
The DSE test is fine, and I think that you should keep it, but can you please add a direct AA test (like, e.g., test/Analysis/BasicAA/cs-cs.ll)?
Should we just always return ModRefInfo::Ref in this case? It can read the data (to copy it), but can't write, correct?
https://reviews.llvm.org/D50679
More information about the llvm-commits
mailing list