[PATCH] D126641: [RISCV] Pass OptLevel to `RISCVDAGToDAGISel` correctly

Yueh-Ting (eop) Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 30 17:27:17 PDT 2022


eopXD added a comment.

In D126641#3546557 <https://reviews.llvm.org/D126641#3546557>, @eopXD wrote:

> In D126641#3546498 <https://reviews.llvm.org/D126641#3546498>, @craig.topper wrote:
>
>> I see that this is how many other targets do it. It would have been helpful to mention that in the description so we immediately know you're not inventing something new.
>>
>> LGTM
>
> Yes you are right, I could have done better explaining my patch.
> I will try to mention how other targets do to bring in the context.
> Thank you for the swift review.

What I meant is I will explain things better in the future.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126641



More information about the llvm-commits mailing list