[PATCH] D27131: [ELF] - Add support of proccessing of the rest allocatable synthetic sections from linkerscript.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 27 23:58:59 PST 2016


grimar added a comment.

Testcase has next script:

  SECTIONS { . = SIZEOF_HEADERS; .foo : { *(.*) } : text : foo}

That means it puts all sections to foo, and since this patch added few more synthetic sections available from script,
it put it there and total size grew up.



================
Comment at: test/ELF/eh-frame-marker.s:6-7
 
+// CHECK:      Name: .eh_frame_hdr
 // CHECK:      Name: .eh_frame
 // CHECK-NEXT: Type: SHT_PROGBITS
----------------
emaste wrote:
> I don't think it should matter, but having these two sections ordered this way is consistent with what the GNU linkers do.
I did not try to check their relative order here. I think it is not what is important.
Problem is that line

```
// CHECK:      Name: .eh_frame
```

accepts foth .eh_frame and .eh_frame_hdr. I did not find the way to do exact matching. So first line here is just to skip the hdr section.


https://reviews.llvm.org/D27131





More information about the llvm-commits mailing list