[PATCH] D23908: LSV: Always try to adjust the alloca alignment

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 29 12:41:57 PDT 2016


jlebar added a comment.

accessIsMisaligned returns false if the TTI reports that the access is not "fast".  But here we seem to be ignoring that and aligning the access anyway.

Is there prior art for ignoring TTI and assuming that unaligned accesses are slower than aligned accesses?  Alternatively, could you just make your target return false for "fast"?


https://reviews.llvm.org/D23908





More information about the llvm-commits mailing list