[PATCH] D59404: [ELF] Add a test for large .bss.rel.ro

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 14 20:10:05 PDT 2019


pcc added inline comments.


================
Comment at: test/ELF/relro-bss.s:8
+# SEGMENT:      .text
+# SEGMENT-NEXT: .data.rel.ro .bss.rel.ro
+# SEGMENT-NEXT: .data .bss
----------------
I'd test the program headers explicitly here, so that you can be sure that all of the fields are correct.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D59404





More information about the llvm-commits mailing list