diff options
| author | Xiangyang (Mark) Guo <helloguo@fb.com> | 2024-03-28 10:11:57 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-03-28 10:11:57 -0700 |
| commit | 1607e8212caa7c050d89b5b143531edb815ec7f3 (patch) | |
| tree | cfc3107bb4eaf558bea864a7b6c47530a41137b0 /llvm/lib/Analysis/InlineCost.cpp | |
| parent | 6aee1f9d1833f7ac000f6451a5d440b3a321a557 (diff) | |
| download | llvm-1607e8212caa7c050d89b5b143531edb815ec7f3.zip llvm-1607e8212caa7c050d89b5b143531edb815ec7f3.tar.gz llvm-1607e8212caa7c050d89b5b143531edb815ec7f3.tar.bz2 | |
[InlineCost] Disable cost-benefit when sample based PGO is used (#86626)
#66457 makes InlineCost to use cost-benefit by default, which causes
0.4-0.5% performance regression on multiple internal workloads. See
discussions https://github.com/llvm/llvm-project/pull/66457. This pull
request reverts it.
Co-authored-by: helloguo <helloguo@meta.com>
Diffstat (limited to 'llvm/lib/Analysis/InlineCost.cpp')
| -rw-r--r-- | llvm/lib/Analysis/InlineCost.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Analysis/InlineCost.cpp b/llvm/lib/Analysis/InlineCost.cpp index e55eaa5..c75460f 100644 --- a/llvm/lib/Analysis/InlineCost.cpp +++ b/llvm/lib/Analysis/InlineCost.cpp @@ -800,7 +800,7 @@ class InlineCostCallAnalyzer final : public CallAnalyzer { return false; } else { // Otherwise, require instrumentation profile. - if (!(PSI->hasInstrumentationProfile() || PSI->hasSampleProfile())) + if (!PSI->hasInstrumentationProfile()) return false; } |