[PATCH] D127581: [ELF] Relax R_RISCV_ALIGN

[Luís Marques via Phabricator via llvm-commits]

luismarques added inline comments.


================
Comment at: lld/ELF/LinkerScript.cpp:949-950
 
 // This function assigns offsets to input sections and an output section
 // for a single sections command (e.g. ".text { *(.text); }").
+bool LinkerScript::assignOffsets(OutputSection *sec) {
----------------
Document return


================
Comment at: lld/ELF/LinkerScript.cpp:1326
     }
-    assignOffsets(&cast<OutputDesc>(cmd)->osec);
+    addrChanged |= assignOffsets(&cast<OutputDesc>(cmd)->osec);
   }
----------------
We don't have test coverage for this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127581/new/

https://reviews.llvm.org/D127581