[PATCH] D65148: [SimplifyCFG] Bump phi-node-folding-threshold from 2 to 3

Wed Aug 14 10:52:34 PDT 2019

lebedev.ri added a comment.

In D65148#1629103 <https://reviews.llvm.org/D65148#1629103>, @lebedev.ri wrote:

> In D65148#1629010 <https://reviews.llvm.org/D65148#1629010>, @dmgreen wrote:
>
> > > That is a valid transformation: https://rise4fun.com/Alive/bxq
> >
> > I see, because of the truncs back into an i16. That makes sense.
> >
> > > So i suppose my question here is - other than this, did you observe any other regressions?
> > >  If not, care to tentatively stamp? I will *not* land this patch until after that last needed patch.
> >
> > It's hard to tell for sure if they are all the same thing.
>

Posted D66232 <https://reviews.llvm.org/D66232>, it finishes cleaning `CLAMP` pattern.
As far i'm currently aware that is the last missing bit from my side..

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65148