[PATCH] D149166: [RISCV] Try to perform select => binop combines when conditional operations are available

[Alex Bradbury via Phabricator via llvm-commits]

asb added a comment.

In D149166#4297768 <https://reviews.llvm.org/D149166#4297768>, @craig.topper wrote:

> I had wondered if it made sense to handle Zicond/XventanaCondOps in LowerSelect by creating RISCV::CZERO_EQZ/CZERO_NEZ opcodes. Would cut down on the number of isel patterns.

That's not an option I'd explored, but I like that it might provide more uniformity for the condops vs non-condops. Let me have a play with that alternative.


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

https://reviews.llvm.org/D149166