[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