[PATCH] D80229: [Inlining] Set inline-deferral-scale to 2.

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

I finally understood what this code is doing ... and I'm wondering why you only tested scales from 2 upwards, and did not test a scale of 1. Scale 1 should be the one that is actually "correct" in terms of modeling, as we are choosing to incur TotalCost in place of IC.getCost().


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80229