[PATCH] D82721: [ScalarEvolution] createSCEV(): recognize `udiv`/`urem` disguised as an `sdiv`/`srem`

Wed Jul 1 17:50:42 PDT 2020

efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM

================
Comment at: llvm/test/Analysis/ScalarEvolution/sdiv.ll:17
 ; CHECK-NEXT:    %rem = sdiv i32 %i.0, 2
-; CHECK-NEXT:    --> (zext i1 {false,+,true}<%for.cond> to i32) U: [0,2) S: [0,2) Exits: (zext i1 (trunc i32 %width to i1) to i32) LoopDispositions: { %for.cond: Computable }
 ; CHECK-NEXT:    %idxprom = sext i32 %rem to i64
----------------
clementval wrote:
> lebedev.ri wrote:
> > efriedma wrote:
> > > What are you diffing against here?
> > Didn't push the test commit originally.
> > Now done: rG51ff7642a33f73518d60909e3fe4e6348dcc7b27.
> This commit is making a bunch of builedbot failing. Can you revert it?
Oh, that makes more sense.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82721