[PATCH] D32289: [ELF] - Linkerscript: support combination of linkerscript and --compress-debug-sections.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 01:16:35 PDT 2017


>Rui Ueyama <ruiu at google.com> writes:
>>> In the most general case (like for thunks), we will have to run
>>> assignAddresses multiple times.
>>>
>>> For debug info it should be trivial to change processCommands to compute
>>> the offsets.
>>>
>>> BTW, should we avoid compressing or error out if the linker script has a
>>> symbol assignment? For example:
>>>
>>> SECTIONS {
>>> .debug_str : { *(foo) sym=.; *(bar) }
>>> }
>>>
>>
>> Is this a hypothetical use case, or do you actually have something like
>> this?
>
>Hypothetical. When I wrote that it would be trivial to change
>processCommands I wondered where that would fail.
>
>Cheers,
>Rafael

When I started writting this patch I searched for scripts in the web,
but did not find any which do anything like this for debug sections.

May be we should always error out if user tries layouting the debug sections ?
This looks more like a error/mistype than intentional change it seems.

Best regards,
George | Developer | Access Softek, Inc


More information about the llvm-commits mailing list