[llvm] r293936 - Change how we handle section symbols on ELF.

Mikael Holmén via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 10 07:13:47 PST 2017


On 02/10/2017 04:05 PM, Rafael Espíndola wrote:
> Can you check if the attached patch fixes the issue for you?
>

Yep, now my function "code" is accepted!

Thanks,
Mikael

> Cheers,
> Rafael
>
>
> On 10 February 2017 at 09:05, Rafael Avila de Espindola
> <rafael.espindola at gmail.com> wrote:
>> Mikael Holmén <mikael.holmen at ericsson.com> writes:
>>
>>> Hi,
>>>
>>> On 02/10/2017 02:38 PM, Rafael Avila de Espindola wrote:
>>>> Sorry, I missed your first reply. Taking a look at the xcore testcase.
>>>
>>> Great! I guessed so.
>>>
>>> One could of course argue that there can't be functions on C/C++ level
>>> starting with "." so the xcore crash can't happen in the real world but
>>> that would be really sad for us since our section is actually called "code".
>>
>> The function name is valid in llvm, so it is a bug in llvm even if it is
>> not visible from c++.
>>
>> Cheers,
>> Rafael



More information about the llvm-commits mailing list