[llvm] r300205 - LTO: Pass SF_Executable flag through to InputFile::Symbol

Tobias Edler von Koch via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 10:57:37 PDT 2017


On 04/13/2017 12:51 PM, Davide Italiano wrote:
>> [...]
>>
>>         if (Sym.isWeak() && Sym.isIndirect())
>> -        outs() << "        fallback " << Sym.getCOFFWeakExternalFallback() << '\n';
>> +        outs() << "         fallback " << Sym.getCOFFWeakExternalFallback() << '\n';
>>       }
>>
>>       outs() << '\n';
> These seem to be unrelated formatting changes. Are them intended?


Yes, they are. It's meant to be indented to match the width of the flags 
on previous lines (e.g. "D------X"). With the addition of Executable, 
those are now one character wider.

Thanks,
Tobias

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project.



More information about the llvm-commits mailing list