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

Wed Jun 30 12:22:27 PDT 2021

mstorsjo added a comment.

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

> @mstorsjo @uabelho  could you check if e6d22d0174e0 <https://reviews.llvm.org/rGe6d22d0174e09fa01342d9ed1dca47bc1eb58303> fixes the issues you are seeing? I checked with @mstorsjo's example and with the new patch we get the same results as with D99424 <https://reviews.llvm.org/D99424> reverted.

Thanks! It does seem like that commit fixes the issues I was observing in both those two testcases.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99424