[llvm] Reland "[ObjectYAML][ELF] Take alignment into account when generating notes" (PR #118434)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 4 00:28:00 PST 2024


================
@@ -1810,7 +1810,7 @@ void ELFState<ELFT>::writeSectionContent(Elf_Shdr &SHeader,
     break;
   default:
     reportError(Section.Name + ": invalid alignment for a note section: 0x" +
-                Twine::utohexstr(SHeader.sh_addralign));
+                Twine::utohexstr(Section.AddressAlign));
----------------
jh7370 wrote:

Possibly not worth it, but I wonder if a test is in order to highlight which alignment is actually used (i.e. `Section.AddressAlign`, not `SHeader.sh_addralign`)?

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


More information about the llvm-commits mailing list