[PATCH] D136736: [LSR][TTI][RISCV] Add isAllowDropLSRSolution into TTI and enable it for RISC-V

Yueh-Ting (eop) Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 30 19:21:22 PDT 2022


eopXD added a comment.

In D136736#3895242 <https://reviews.llvm.org/D136736#3895242>, @fhahn wrote:

> Could you elaborate why a target-specific option is needed? Shouldn't it be beneficial for all targets and be enabled independently?

Enabling in all targets will require a lot of changes in test cases across targets so I think I should divide and conquer here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136736



More information about the llvm-commits mailing list