[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