[PATCH] D22961: [ELF] - Linkerscript: restrict moving location counter backwards.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 29 07:16:17 PDT 2016


grimar added a comment.

In https://reviews.llvm.org/D22961#500600, @evgeny777 wrote:

> Does anyone really wants this? There should be huge number of ways to make broken image besides this.


Not sure that it is the reason not to try to fix at least something known.

> Also AFAIK the code below is valid:

> 

>   .debug_info     0 : { *(.debug_info .gnu.linkonce.wi.*) }

>   .debug_abbrev   0 : { *(.debug_abbrev) }

> 

> 

> And I wonder if it is different from this one (which, I guess, your patch is making invalid)

> 

>   . = 0;

>   .debug_info  : { *(.debug_info .gnu.linkonce.wi.*) }

>   . = 0;

>   .debug_abbrev  : { *(.debug_abbrev) }

> 


Sections you mentioned are all non alloc I think. We can make an exception for them.
Interesting that gold ignores [address] for non allocatable sections at all.


https://reviews.llvm.org/D22961





More information about the llvm-commits mailing list