[PATCH] D30419: [ELF] - Define __bss_start symbol.

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 14:26:31 PDT 2017


Can you rebase this patch? It should be simpler now that BssSection is
a SyntheticSection, since we should have a single .bss now.

Cheers,
Rafael


On 7 March 2017 at 03:53, George Rimar <grimar at accesssoftek.com> wrote:
>>So is it ok for commit with original check ?
>>>>> +    bool IsBSS = (In<ELFT>::Common && In<ELFT>::Common->OutSec == OS) ||
>>>>> +                 OS == Out::Bss || OS == Out::BssRelRo;
>>
>>George.
>
> Hi Rafael,
>
> so is commiting with this original check (and not NOBITS) is fine for you ?
>
> George.


More information about the llvm-commits mailing list