[PATCH] D90113: [DAGCombiner] Fold BinOp into Select containing identity constant

Tue Feb 8 05:16:23 PST 2022

spatel added a comment.

In D90113#3282854 <https://reviews.llvm.org/D90113#3282854>, @RKSimon wrote:

> @laytonio Hasn't been active since Dec 2020 - is there anyone that has the bandwidth to continue this (either comandeer or start a new patch)?

We're not there yet, but we're on a path to eventually overlap this patch with:
D118644 <https://reviews.llvm.org/D118644> (enabled this fold for a subset of x86 vector types/opcodes) 
D119150 <https://reviews.llvm.org/D119150> (moves the code into DAGCombiner with a TLI hook)
D119111 <https://reviews.llvm.org/D119111> (adds more FP opcodes - abandoned, but we can easily adapt it into D119150 <https://reviews.llvm.org/D119150>)

This first set of patches is a direct response to perf regressions that occurred via D113442 <https://reviews.llvm.org/D113442>, so we will try to get them into the 14 release branch too.

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

https://reviews.llvm.org/D90113