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

[Mikael Holmén via Phabricator via llvm-commits]

uabelho added a comment.

I've also bisected a miscompile to this patch. I don't know at all what is happening but the suggested fix

  -        Scale = APInt::getOneBitSet(Scale.getBitWidth(),
  -                                    Scale.countTrailingZeros());
  +        GCD = APInt(Scale.getBitWidth(), 1);

makes the miscompile go away.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99424