[LLD][ELF] Add linker script generated data to non-contiguous relro test [NFC].

[peter.smith]

Summary

Adapted from an example in PR35478. If in the future we are going to ignore zero sized sections we need to account for linker scripts with location counter expressions or data creating functions such as BYTE. These linker script commands may be evaluated late on so testing early for OutputSection size will miss them.

This should only be committed once the revert of r318924 has been done (D40710).

[jhenderson]

test/ELF/relro-non-contiguous.s

27–28 ↗

(On Diff #125148)

I feel like these should be in two separate links. By combining the two, if one is correctly seen as influencing the size, but not the other, we won't notice.

[peter.smith]

Thanks for the comment, I've uploaded a new diff that tests each case individually.

[peter.smith]

This is almost exactly what I added as
test/ELF/relro-non-contiguous-script-data.s.

Would you mind rebasing this to add the BYTE case there instead?

Sorry,
Rafael

Sure I'll just commit that as it will be just a small addition to the existing test.

lld/trunk/test/ELF/relro-non-contiguous-script-data.s

 // REQUIRES: x86
 
 // RUN: echo "SECTIONS { \
 // RUN: .dynamic : { *(.dynamic) } \
 // RUN: .non_ro : { . += 1; } \
 // RUN: .jcr : { *(.jcr) } \
 // RUN: } " > %t.script
 // RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %s -o %t.o
 // RUN: not ld.lld --export-dynamic %t.o -o %t --script=%t.script 2>&1 | FileCheck %s
 
+// RUN: echo "SECTIONS { \
+// RUN: .dynamic : { *(.dynamic) } \
+// RUN: .non_ro : { BYTE(1); } \
+// RUN: .jcr : { *(.jcr) } \
+// RUN: } " > %t2.script
+// RUN: not ld.lld --export-dynamic %t.o -o %t --script=%t2.script 2>&1 | FileCheck %s
+
 // CHECK: error: section: .jcr is not contiguous with other relro sections
 
 .global _start
 _start:
 
         // non-empty relro section
         .section .jcr, "aw", @progbits
         .quad 0