[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