[PATCH] D73217: [InlineCost] Add flag to allow changing the default inline cost

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 22 12:24:01 PST 2020


davidxl added a comment.

This change won't work. See

  if (InlineThreshold.getNumOccurrences() == 0) {
    Params.OptMinSizeThreshold = InlineConstants::OptMinSizeThreshold;
    Params.OptSizeThreshold = InlineConstants::OptSizeThreshold;
    Params.ColdThreshold = ColdThreshold;
  } else if (ColdThreshold.getNumOccurrences() > 0) {
    Params.ColdThreshold = ColdThreshold;
  }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73217





More information about the llvm-commits mailing list