[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:22:32 PDT 2022


eopXD added a comment.

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.


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