[PATCH] D108220: [RISCV] Let lowerSELECT merge SETCC into SELECT_CC for floats

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 17 11:29:26 PDT 2021


craig.topper added a comment.

Do we have test coverage for this fp selects with integer conditions? The DAG combine was added in https://reviews.llvm.org/D98132 but I don't see any FP related test changes in there. I'm not sure I realized I was changing the behavior of FP selects at the time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108220



More information about the llvm-commits mailing list