[PATCH] D86127: [Attributor] Make AAMemoryLocationCallSite behavior sane

Stefan Stipanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 9 23:58:41 PDT 2020


sstefan1 added a comment.

Sorry for the delay.

In D86127#2223069 <https://reviews.llvm.org/D86127#2223069>, @jdoerfert wrote:

> An explicit test will be added.

Are you going to add this test?



================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:6235
+    // AAMemoryLocation to describe the accessed location within the caller
+    // scope. If we set the know argmemonly bit it behaves differently than
+    // the argmemonly bit on a store next. There are two downsides, one
----------------
typo: known


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86127/new/

https://reviews.llvm.org/D86127



More information about the llvm-commits mailing list