[PATCH] D99674: [InstCombine] Conditionally fold select i1 into and/or

Sat Apr 3 21:35:19 PDT 2021

aqjune added a comment.

In D99674#2667634 <https://reviews.llvm.org/D99674#2667634>, @nikic wrote:

> I'm OK with landing this first to fix an active miscompile, but I don't think we should block flipping the switch on noundef work. I think your numbers support that we shouldn't wait on that, because the noundef impact it small relative to the whole change (it only makes a 15% difference in affected files).

Let's wait for two weeks after this patch is landed to see whether any serious regression from this patch is reported - if there is no such report, I think it is a good sign and we can move forward without waiting for the noundef patch.
Thanks for the review!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99674