[PATCH] D26803: Fix known zero bits for addrspacecast

Yaxun Liu via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 07:33:40 PST 2016


yaxunl added a comment.

In https://reviews.llvm.org/D26803#599068, @arsenm wrote:

> LGTM. I think it should be ok to preserve the alignment though


I think we may need another patch for that. Basically we may need a dedicated parameter for the known zero bits from alignment and pass it through addrspacecast.



================
Comment at: test/Analysis/ValueTracking/knownzero-addrspacecast.ll:25
+}
\ No newline at end of file

----------------
mcrosier wrote:
> Please add a newline.
will do when committing.


https://reviews.llvm.org/D26803





More information about the llvm-commits mailing list