[llvm] [InstCombine] Fold "extract (select (cond, insert(agg, elem), FV))" (PR #115969)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 14 01:43:37 PST 2024


nikic wrote:

> > > We may generalize to use simplifyInstruction for other instructions in the future.
> > 
> > 
> > I'd rather directly do that than add a special case for extractelement. It's easier to generalize this than not. I did a quick test at #116073 and it doesn't seem to affect compile-time, which would have been my main concern.
> 
> @nikic Is it okay to just keep these negative tests and make this patch NFC? Positive tests have been added in [2ca25ab](https://github.com/llvm/llvm-project/commit/2ca25ab11d01ceacf359643b09aed7d53d0ff8dc).

Sure

https://github.com/llvm/llvm-project/pull/115969


More information about the llvm-commits mailing list