Addressing reviewers.

Author: paschalis-mpeis

bolt/lib/Passes/LongJmp.cpp

@@ -11,7 +11,6 @@
 //===----------------------------------------------------------------------===//
 
 #include "bolt/Passes/LongJmp.h"
-#include "bolt/Utils/CommandLineOpts.h"
 
 #define DEBUG_TYPE "longjmp"
 
@@ -326,7 +325,7 @@ uint64_t LongJmpPass::tentativeLayoutRelocMode(
     const BinaryContext &BC, std::vector<BinaryFunction *> &SortedFunctions,
     uint64_t DotAddress) {
   // Compute hot cold frontier
-  uint32_t LastHotIndex = -1u;
+  int64_t LastHotIndex = -1u;
   uint32_t CurrentIndex = 0;
   if (opts::HotFunctionsAtEnd) {
     for (BinaryFunction *BF : SortedFunctions) {
@@ -351,23 +350,21 @@ uint64_t LongJmpPass::tentativeLayoutRelocMode(
   // Hot
   CurrentIndex = 0;
   bool ColdLayoutDone = false;
+  auto runColdLayout = [&]() {
+    DotAddress = tentativeLayoutRelocColdPart(BC, SortedFunctions, DotAddress);
+    ColdLayoutDone = true;
+    if (opts::HotFunctionsAtEnd)
+      DotAddress = alignTo(DotAddress, opts::AlignText);
+  };
   for (BinaryFunction *Func : SortedFunctions) {
     if (!BC.shouldEmit(*Func)) {
       HotAddresses[Func] = Func->getAddress();
-      // Don't perform any tentative address estimation of a function's cold
-      // layout if it won't be emitted, unless we are ignoring a large number of
-      // functions (ie, on lite mode) and we haven't done such estimation yet.
-      if (opts::processAllFunctions() || ColdLayoutDone)
-        continue;
-    }
-    if (!ColdLayoutDone && CurrentIndex >= LastHotIndex) {
-      DotAddress =
-          tentativeLayoutRelocColdPart(BC, SortedFunctions, DotAddress);
-      ColdLayoutDone = true;
-      if (opts::HotFunctionsAtEnd)
-        DotAddress = alignTo(DotAddress, opts::AlignText);
+      continue;
     }
 
+    if (!ColdLayoutDone && CurrentIndex >= LastHotIndex)
+      runColdLayout();
+
     DotAddress = alignTo(DotAddress, Func->getMinAlignment());
     uint64_t Pad =
         offsetToAlignment(DotAddress, llvm::Align(Func->getAlignment()));
@@ -386,9 +383,11 @@ uint64_t LongJmpPass::tentativeLayoutRelocMode(
     ++CurrentIndex;
   }
 
+  // Ensure that tentative code layout always runs for cold blocks.
   if (!ColdLayoutDone)
-    assert(ColdLayoutDone &&
-           "Did not perform tentative code layout for cold blocks.");
+    runColdLayout();
+  assert(ColdLayoutDone &&
+         "Did not perform tentative code layout for cold blocks.");
 
   // BBs
   for (BinaryFunction *Func : SortedFunctions)

bolt/test/AArch64/split-funcs-lite.test

@@ -9,6 +9,6 @@ REQUIRES: system-linux
 RUN: %clang %cflags %p/../Inputs/asm_main.c -Wl,-q -o %t
 
 RUN: llvm-bolt %t -o %t.bolt -lite=1 -split-functions -split-all-cold \
-RUN: --skip-funcs="_init,_start,call_weak_fn/1,deregister_tm_clones/1,register_tm_clones/1,__do_global_dtors_aux/1,frame_dummy/1,main,foo,_fini" 2>&1 | FileCheck %s
+RUN: --skip-funcs="main,foo" 2>&1 | FileCheck %s
 
 CHECK-NOT: Did not perform tentative code layout for cold blocks.