[llvm] [AArch64][BOLT] Ensure tentative code layout for cold BBs runs. (PR #96609)

Paschalis Mpeis via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 2 03:03:54 PDT 2024


================
@@ -382,6 +382,13 @@ uint64_t LongJmpPass::tentativeLayoutRelocMode(
     DotAddress += Func->estimateConstantIslandSize();
     ++CurrentIndex;
   }
+
+  // Ensure that tentative code layout always runs for cold blocks.
+  if (!ColdLayoutDone)
+    runColdLayout();
+  assert(ColdLayoutDone &&
----------------
paschalis-mpeis wrote:

True. I kept it just for testing but TBH I also did not like it.

This is now gone. I'm using a mock profile to trigger this, and I check whether any cold tentative estimation happened.

I've also updated the [description of this PR](https://github.com/llvm/llvm-project/pull/96609#issue-2372128097).

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


More information about the llvm-commits mailing list