[PATCH] D12853: Introduce !value.align metadata for load instruction

[Artur Pilipenko via llvm-commits]

apilipenko added inline comments.

================
Comment at: lib/Analysis/ValueTracking.cpp:2962
@@ -2956,3 +2961,3 @@
   if (!BaseAlign) {
     Type *Ty = Base->getType()->getPointerElementType();
     BaseAlign = DL.getABITypeAlignment(Ty);
----------------
sanjoy wrote:
> With your changes, does this bit remain necessary?  I think this part is unsound anyway (since it relies on differentiating different pointer types of the same address space).
I'm going to remove this in a separate patch.


http://reviews.llvm.org/D12853