There are a variety of regressions at the moment, but it would be nice
if we could turn this on in the backend effectively so that
InstCombine can generically fold (select c, BinOp(X, Y), X) ->
BinOp(X, (select c, Y, Indentity)) which is generally preferable in
the middle-end.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Time | Test | |
---|---|---|
150 ms | x64 debian > LLVM.tools/llvm-locstats::locstats.ll |
Event Timeline
llvm/test/CodeGen/X86/avx512-intrinsics-fast-isel.ll | ||
---|---|---|
1889–1893 | Is this change intentional? | |
llvm/test/CodeGen/X86/bool-simplify.ll | ||
56–60 | This looks regression to me. ditto below. | |
llvm/test/CodeGen/X86/bool-vector.ll | ||
68 | Introducing many branches, which doesn't look good. | |
llvm/test/CodeGen/X86/fold-select.ll | ||
10–16 | Regression? ditto below. |
llvm/test/CodeGen/X86/fold-select.ll | ||
---|---|---|
10–16 | Yeah mostly regressions. Working on trying to clean it up. |
llvm/test/tools/llvm-locstats/locstats.ll | ||
---|---|---|
1 ↗ | (On Diff #523424) | Is this necessary? |
Is this change intentional?