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

[Yueh-Ting (eop) Chen via Phabricator via llvm-commits]

eopXD added a comment.

In D136736#3895386 <https://reviews.llvm.org/D136736#3895386>, @eopXD wrote:

> 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.

@fhahn Any more comments before merging this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136736