[llvm] r236642 - MC: Skip names of temporary symbols in object streamer

Rafael EspĂ­ndola rafael.espindola at gmail.com
Tue Jun 2 16:00:32 PDT 2015


>> On 2015-May-21, at 12:17, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
>>
>>> +  if (IsTemporary && AlwaysAddSuffix && !UseNamesOnTempLabels)
>>> +    return new (*this) MCSymbol("", true);
>>
>> Why do you need the AlwaysAddSuffix?
>
>
> IIRC, I tried it without and got some test failures; I assume some
> symbols make it to the object file with `L` or `l` prefixes and need
> to be stripped by the linker.  I didn't look too deeply, since this
> got me the same memory win (there must not be many temp symbols with
> `!AlwaysAddPrefix`).

You were in the right path. I clarified a bit which temp symbol
currently need a name in r238887.

Cheers,
Rafael




More information about the llvm-commits mailing list