[llvm] [BOLT] Add structure of CDSplit to SplitFunctions (PR #73430)

Maksim Panchenko via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 27 16:20:45 PST 2023


================
@@ -409,8 +487,10 @@ void SplitFunctions::splitFunction(BinaryFunction &BF, SplitStrategy &S) {
     LLVM_DEBUG(dbgs() << "Estimated size for function " << BF
                       << " post-split is <0x" << Twine::utohexstr(HotSize)
                       << ", 0x" << Twine::utohexstr(ColdSize) << ">\n");
-    if (alignTo(OriginalHotSize, opts::SplitAlignThreshold) <=
-        alignTo(HotSize, opts::SplitAlignThreshold) + opts::SplitThreshold) {
+    if (S.autoReversal() &&
----------------
maksfb wrote:

Do you mean to run `calculateEmittedSize()` above even when `autoReversal()` is off?

https://github.com/llvm/llvm-project/pull/73430


More information about the llvm-commits mailing list