[PATCH] D150510: [ELF] x86-64: place .lrodata, .lbss, and .ldata away from code sections

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: lld/test/ELF/x86-64-section-layout.s:24
+# CHECK-NEXT:  .ldata    PROGBITS        0000000000203307 000307 000002 00 WAl  0   0  1
+# CHECK-NEXT:  .ldata2   PROGBITS        0000000000203309 000309 000001 00 WAl  0   0  1
+# CHECK-NEXT:  .lbss     NOBITS          000000000020330a 00030a 000002 00 WAl  0   0  1
----------------
aeubanks wrote:
> shouldn't this be merged with `.ldata` according to the change in `LinkerScript.cpp`?
No. `.ldata2` is picked to test that `.ldata2` is not combined into `.ldata .ldata.*` sections.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150510