[PATCH] D91027: [BasicAA] Generalize base offset modulus handling

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 9 10:17:58 PST 2020

Previous message: [PATCH] D91027: [BasicAA] Generalize base offset modulus handling
Next message: [PATCH] D91081: [flang] Avoid calling the linker when "-c" option is used
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

jdoerfert added a comment.

Yes, now I get it. I think I was also missing the "V2 has offset 0" precondition.

I like this, nice cleanup that adds more power. I think it is correct and good to go, would prefer a second opinion though.


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

https://reviews.llvm.org/D91027

Previous message: [PATCH] D91027: [BasicAA] Generalize base offset modulus handling
Next message: [PATCH] D91081: [flang] Avoid calling the linker when "-c" option is used
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list