[PATCH] D79268: [RISCV] Implement Hooks to avoid chaining SELECT

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 1 17:18:12 PDT 2020


lenary updated this revision to Diff 261569.
lenary added a comment.

@luismarques pointed out there's an easier implementation of this: to use
`setHasMultipleConditionRegisters`, instead of overriding
`shouldNormalizeToSelectSequence`. This is correct as RISC-V branches can use
any register they want.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79268

Files:
  llvm/lib/Target/RISCV/RISCVISelLowering.cpp
  llvm/test/CodeGen/RISCV/double-previous-failure.ll
  llvm/test/CodeGen/RISCV/select-or.ll
  llvm/test/CodeGen/RISCV/zext-with-load-is-free.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79268.261569.patch
Type: text/x-patch
Size: 5436 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200502/25e9b3ba/attachment-0001.bin>


More information about the llvm-commits mailing list