[PATCH] D86243: [InstCombine] canonicalize 'not' ops before logical shifts

Thu Aug 20 06:59:42 PDT 2020

bjope added a comment.

In D86243#2227038 <https://reviews.llvm.org/D86243#2227038>, @xbolva00 wrote:

> Some SCEV/pipeline tests?
>
> Maybe from https://reviews.llvm.org/D85969#change-8DkHYel6Y1JK ?

I figure we want to move forward with D85969 <https://reviews.llvm.org/D85969> and land that as well. Right?
I mean, can't ScalarEvolution analysis be used by some pass before instcombine in the pass pipeline. Then I assume it is nice if SCEV treats the two different forms of the equivalent expressions the same way.

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

https://reviews.llvm.org/D86243