[PATCH] D130809: [RISCV] Explicitly select second operand of branch condition to X0.

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 31 22:12:06 PDT 2022


asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.

I'm obviously a big fan of the codegen improvements resulting from this patch...but it always feels unpleasant to add a target-specific workaround for what seems to be a bug/limitation elsewhere. I think it does make sense to land this workaround as it's not clear how feasible improving the coalescer would be, but perhaps you could file an issue to track this problem?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130809



More information about the llvm-commits mailing list