[PATCH] D28394: [AliasAnalysis] Limit `MemoryLocation` retrieval to valid intrinsic arguments.

Finkel, Hal J. via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 10 09:32:51 PST 2017


Yes, I think that's a good idea.

-Hal

Sent from my Verizon Wireless 4G LTE DROID
On Jan 10, 2017 11:13 AM, bryant via Phabricator <reviews at reviews.llvm.org> wrote:
bryant added a comment.

Should invariant.start be updated to `declare nolias {}* @llvm.invariant.start.p0i8(i64 <size>, i8* nocapture <ptr>)`? That way, the returned token is ensured not to mess with AA queries.


Repository:
  rL LLVM

https://reviews.llvm.org/D28394



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170110/2b9ed9b3/attachment.html>


More information about the llvm-commits mailing list