[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