[PATCH] D99424: [BasicAA] Be more careful with modulo ops on VariableGEPIndex.

Wed Jun 30 03:16:11 PDT 2021

fhahn added a comment.

In D99424#2849581 <https://reviews.llvm.org/D99424#2849581>, @fhahn wrote:

> In D99424#2849458 <https://reviews.llvm.org/D99424#2849458>, @mstorsjo wrote:
>
>> The problem appears with https://martin.st/temp/pixlet-preproc.c, compiled with “clang -target aarch64-w64-mingw32 -c -O3 pixlet-preproc.c”. I haven’t pinpointed exactly what changes and whether that’s wrong or if the source itself relies on something undefined though, or if there’s some strict aliasing violation.

If you manage to reduce it further down, that would be very helpful. Unfortunately it looks like the only difference comes from the MachineScheduler, so it is a bit tricky to extract a nice test case.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99424