This removes selectSETCC and adds isel patterns for seteq/setne
conditions.
This removes the duplication of selectSETCC between lowering and
isel. This also gets some cases in xaluo.ll that we missed previously.
Paths
| Differential D156250
[RISCV] Handle seteq/setne conditions for CZERO_NEZ/CZERO_EQZ during isel. ClosedPublic Authored by craig.topper on Jul 25 2023, 10:11 AM.
Details Summary This removes selectSETCC and adds isel patterns for seteq/setne This removes the duplication of selectSETCC between lowering and
Diff Detail
Event TimelineComment Actions LGTM in general.
This revision is now accepted and ready to land.Jul 25 2023, 8:27 PM
This revision was landed with ongoing or failed builds.Jul 26 2023, 10:06 AM Closed by commit rGe28307e93adb: [RISCV] Handle seteq/setne conditions for CZERO_NEZ/CZERO_EQZ during isel. (authored by craig.topper). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 544417 llvm/lib/Target/RISCV/RISCVISelLowering.cpp
llvm/lib/Target/RISCV/RISCVInstrInfoXVentana.td
llvm/lib/Target/RISCV/RISCVInstrInfoZicond.td
llvm/test/CodeGen/RISCV/condops.ll
llvm/test/CodeGen/RISCV/xaluo.ll
|
These patterns are almost the same as zicond's, is it possible to remove the redundancy?
(Just a weak suggestion, it's OK to ignore it)