[PATCH] D102552: [GlobalOpt] recompute alignments for loads and stores of updated globals

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 19 09:01:00 PDT 2021


spatel added a comment.

In D102552#2768607 <https://reviews.llvm.org/D102552#2768607>, @lebedev.ri wrote:

> Thinking about it more, i guess not having an obvious miscompile would be a good starting point.
> We can deal with padding, if need be, later.

Thanks! I have no sense of whether this is a common/impactful transform, but yes, the fuzzers found this hole in the logic using a simple C program, so we should get it fixed.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102552/new/

https://reviews.llvm.org/D102552



More information about the llvm-commits mailing list