On 26 December 2012 17:53, Weiming Zhao <[email protected]> wrote:
> Hi Rafael,
>
> Thanks for the review.
> I'm attaching the patch which includes test case.

Thanks for adding a test for the warning. I think the patch still
needs a codegen patch that checks that the value passed to -mabi is
actually used.

On the implementation itself: Why do you need to patch the triple
instead of just changing the code using it to give preference to
Opts.ABI?

+      Opts.Triple = T.getTriple();

This just copies the triple back, no?

> Thanks,
> weiming

Cheers,
Rafael
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to