[cfe-commits] Fix for Bug 10978
Eli Friedman
eli.friedman at gmail.com
Fri Sep 30 08:37:42 PDT 2011
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.
@@ -257,21 +258,20 @@ namespace {
}
};
}
-
const TargetAttributesSema &Sema::getTargetAttributesSema() const {
if (TheTargetAttributesSema)
return *TheTargetAttributesSema;
Random whitespace change?
Otherwise, looks fine.
-Eli
More information about the cfe-commits
mailing list