[lld] r193556 - [PECOFF] Set section name for BSS atoms.
Shankar Easwaran
shankare at codeaurora.org
Mon Oct 28 21:26:12 PDT 2013
On 10/28/2013 11:20 PM, Rui Ueyama wrote:
> On Mon, Oct 28, 2013 at 9:07 PM, Shankar Easwaran
> <shankare at codeaurora.org>wrote:
>
>> On 10/28/2013 6:26 PM, Rui Ueyama wrote:
>>
>>> - : COFFDefinedFileAtom(file, name, "", scope, typeZeroFill, perms,
>>> + : COFFDefinedFileAtom(file, name, ".bss", scope, typeZeroFill,
>>> perms,
>>>
>> The section name should be derived by the writer for default section types.
>>
> That is not true for COFF. In COFF, section name has significance unlike
> ELF, and the linker is expected to emit sections with the same name as in
> the object file.
ELF goes by section names too. What I am trying to say here is if the
atoms contentType is typeZeroFill set the section name to .bss. Having a
section name as the default would only increase the native IR file size,
and increase in memory usage.
Thanks
Shankar Easwawran
More information about the llvm-commits
mailing list