On Fri, Sep 30, 2011 at 11:51 AM, Ruben Van Boxem <[email protected]> wrote: > 2011/9/30 Eli Friedman <[email protected]> >> >> 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. > > Fixed. > >> >> @@ -257,21 +258,20 @@ namespace { >> } >> }; >> } >> - >> const TargetAttributesSema &Sema::getTargetAttributesSema() const { >> if (TheTargetAttributesSema) >> return *TheTargetAttributesSema; >> >> Random whitespace change? > > Yes it is, fixed.
r140871. -Eli _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
