[PATCH] D99206: Make FindAvailableLoadedValue TBAA aware

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/include/llvm/Analysis/Loads.h:168
 Value *FindAvailablePtrLoadStore(Value *Ptr, Type *AccessTy, bool AtLeastAtomic,
-                                 BasicBlock *ScanBB,
+                                 Optional<MemoryLocation>, BasicBlock *ScanBB,
                                  BasicBlock::iterator &ScanFrom,
----------------
Why make it optional? The only None use seems to be in JumpThreading, and we should be able to pass a location there as well.

(Could also drop the Ptr argument if it's required, as the memory location includes the pointer.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99206