[llvm] [SystemZ] Remove getInliningThresholdMultiplier() override (PR #94612)
Jonas Paulsson via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 24 01:32:48 PDT 2024
JonPsson1 wrote:
> I'm mainly trying to figure out how important that imagick regression is. If the s390x backend is really going out of its way to make imagick optimize as desired (not just by adjusting the multiplier, but also doing other threshold adjustments), then this is probably too specific to a single benchmark and we should not be overly concerned about it.
I checked back again on imagick, and it seems that removing either the multiplier or the specific threshold adjustment cause a similar regression (~10%). So they are both needed for the current performance. Without the multiplier, the specific adjustment is ~1% improvement only.
https://github.com/llvm/llvm-project/pull/94612
More information about the llvm-commits
mailing list