[cfe-commits] Fix for Bug 10978

[Eli Friedman]

On Fri, Sep 30, 2011 at 11:51 AM, Ruben Van Boxem
<vanboxem.ruben at gmail.com> wrote:
> 2011/9/30 Eli Friedman <eli.friedman at gmail.com>
>>
>> On Fri, Sep 30, 2011 at 2:44 AM, Ruben Van Boxem
>> <vanboxem.ruben at gmail.com> wrote:
>> > http://llvm.org/bugs/show_bug.cgi?id=10978
>> >
>> > Attached is a revised patch that also uses X86AttributesSema for x86_64,
>> > and
>> > rejects the force_align_arg_pointer attribute for x86_64.
>> >
>> > The testcase is just a copy of the existing dllimport-dllexport test,
>> > but I
>> > modified the triple used. I hope this is correct, otherwise, please show
>> > me
>> > how cause I don't know how to set the test options.
>>
>> You can just run the existing test with multiple triples; multiple RUN
>> lines work the way you would expect.
>
> Fixed.
>
>>
>> @@ -257,21 +258,20 @@ namespace {
>>     }
>>   };
>>  }
>> -
>>  const TargetAttributesSema &Sema::getTargetAttributesSema() const {
>>   if (TheTargetAttributesSema)
>>     return *TheTargetAttributesSema;
>>
>> Random whitespace change?
>
> Yes it is, fixed.

r140871.

-Eli