[PATCH] D15401: Refactor threshold computation for inline cost analysis

[Chandler Carruth via llvm-commits]

chandlerc added a comment.

I generally like the direction here, but I think you should actually go further than this.

I think you should change the input to the cost analysis from a numeric threshold to a symbolic enum that selects between the high-level "kinds" of thresholds to use. Then you can internalize all of the logic inside of the cost analysis.

Further, you should feel relatively free to change the behavior of the '-inline-threshold' flag. That flag is primarily a debugging aid. We don't want to flagrantly change its behavior (as that would just be annoying), we don't have any firm or hard contract around exactly how that flag is interpreted. Does that make sense?


Repository:
  rL LLVM

http://reviews.llvm.org/D15401