[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