[PATCH] D74736: [ELF] Ignore the maximum of input section alignments for two cases

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 20 07:35:34 PST 2020


MaskRay marked an inline comment as done.
MaskRay added inline comments.


================
Comment at: lld/ELF/LinkerScript.cpp:767
+    // The alignment is ignored.
+    ctx->outSec->addr = pos;
+  } else {
----------------
grimar wrote:
> It is a bit strange to see that `expandMemoryRegions` is now gone from this path.
> Is it correct?
expandMemoryRegions(0) is a no-op.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74736





More information about the llvm-commits mailing list