[PATCH] D71307: [Loads] Handle simple cases with same base pointer with constant offsets in FindAvailableLoadedValue when AA is null.
    David Li via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jan 17 09:34:51 PST 2020
    
    
  
davidxl added inline comments.
================
Comment at: llvm/lib/Analysis/Loads.cpp:457
+      // alias analysis.
+      APInt LoadOffset(DL.getTypeSizeInBits(Ptr->getType()), 0);
+      APInt StoreOffset(DL.getTypeSizeInBits(
----------------
can you guard the change with 
if (!AA) {
}
as we AA, it will be unnecessary.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71307/new/
https://reviews.llvm.org/D71307
    
    
More information about the llvm-commits
mailing list