On Fri, Sep 30, 2011 at 2:44 AM, Ruben Van Boxem
<[email protected]> 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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to