[lld] [lld][ELF] Refresh .eh_frame_hdr cache after cfi jump table relaxation (PR #228218)
Peter Smith via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 2 01:55:19 PDT 2026
================
@@ -1520,16 +1520,19 @@ template <class ELFT> void Writer<ELFT>::finalizeAddressDependentContent() {
if (ctx.arg.randomizeSectionPadding)
randomizeSectionPadding(ctx);
+ bool cfiJumpTablesChanged = false;
----------------
smithp35 wrote:
I've got a mild preference for moving changed out of the loop, and eliminating `cfiJumpTablesChanged`. Not a lot in it, just means that we don't need to repeatedly set cfiJumpTablesChanged to false in the loop.
I suggest letting other reviewers express their preference before changing.
https://github.com/llvm/llvm-project/pull/228218
More information about the llvm-commits
mailing list