[PATCH] D54743: SROA: preserve alignment tags on loads and stores.

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 20 01:45:12 PST 2018


t.p.northover created this revision.
Herald added subscribers: kosarev, hiraditya, mcrosier.

When splitting up an alloca's uses we were dropping any explicit alignment tags on load and store instructions, which means they default to the ABI-required default alignment and this can cause miscompiles if the original value was smaller.

      

While I was there I decided to move the TBAA data into the base class too, since it was already shared.


Repository:
  rL LLVM

https://reviews.llvm.org/D54743

Files:
  llvm/lib/Transforms/Scalar/SROA.cpp
  llvm/test/Transforms/SROA/alignment.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54743.174734.patch
Type: text/x-patch
Size: 7773 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181120/c8134b6f/attachment.bin>


More information about the llvm-commits mailing list