[PATCH] D23906: LSV: Fix incorrectly increasing alignment

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 26 11:02:30 PDT 2016


arsenm added a comment.

In https://reviews.llvm.org/D23906#526422, @efriedma wrote:

> Is there some reason you're not using getOrEnforceKnownAlignment here?


I didn't know about that one. Switching to that mostly works, except it misses the case  (e.g. store_unknown_offset_align1_i32) where it should be OK because the indexing should avoid an unaligned offset because it uses stripPointerCasts instead of GetUnderlyingObject


https://reviews.llvm.org/D23906





More information about the llvm-commits mailing list