[PATCH] D142232: Increase inline threshold multiplier to 11 in nvptx backend.

Jack Kirk via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 10:34:23 PST 2023


JackAKirk marked an inline comment as done.
JackAKirk added a comment.

In D142232#4103019 <https://reviews.llvm.org/D142232#4103019>, @tra wrote:

> Thank you for the details.
>
> So, on the balance, it appears not to hurt the cases where threshold increase does not have material impact on the compilation time, and in relatively few cases where it does affect compilation time, it does provide sufficient performance improvement. I think it's worth it.
>
> Ship it.

OK thanks. I've updated the comment swapping 5->11, so I'm happy for it to be merged now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142232



More information about the llvm-commits mailing list