[PATCH] D106368: [AlignmentFromAssumptions] avoid crash on alignment constant expression

Tue Jul 20 09:20:36 PDT 2021

spatel added subscribers: erichkeane, lebedev.ri.
spatel added a comment.

In D106368#2890603 <https://reviews.llvm.org/D106368#2890603>, @jdoerfert wrote:

> I still believe we should not create alignment annotations for non-power-of-two constants. This is the second time in a few weeks this causes problems and we still play whack-a-mole instead of restricting clang and teaching the verifier.
>
> https://reviews.llvm.org/D94433

Ah, I didn't realize this is the same problem based on the reduced test. Someone familiar with this feature in Clang -- and compatibility with GCC? -- should have a look then ( @lebedev.ri @erichkeane ? ) - I have no idea what this is supposed to do.

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

https://reviews.llvm.org/D106368