r177329 - This code works around what appears to be a bug in another part of clang.

reed kotler rkotler at mips.com
Tue Mar 19 08:02:39 PDT 2013


On 03/19/2013 07:50 AM, Rafael EspĂ­ndola wrote:
>> So, about the testcase. It is the one in llvm.org/pr15538 that needs
>> reducing, right? If that is so I am more than happy to help you reduce
>> it and create a testcase we can add to clang.
> It reduces to:
>
> struct B {
>    B();
> };
> B::B() {
> }
>
> The problem is that you are being passed an alias because of
> -mconstructor-aliases. Since aliases don't have attributes, the
> correct fix is probably to not call SetTargetAttributes since there is
> nothing for it to set attributes on.
>
> Cheers,
> Rafael
Okay. thanks for the help.

I will take care of this today.

Reed





More information about the cfe-commits mailing list