Hi Rafael,

Thanks for the review. 
I'm attaching the patch which includes test case.

Thanks,
weiming

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by 
The Linux Foundation

-----Original Message-----
From: Rafael Espíndola [mailto:[email protected]] 
Sent: Monday, December 17, 2012 11:24 AM
To: [email protected]
Cc: [email protected]
Subject: Re: [cfe-commits] Fix PR 11326: conflicting mabi between -mabi and 
-target options on ARM

This needs a testcase. Please test that the warning fires and the -mabi value 
is used.

On 12 November 2012 12:59, Weiming Zhao <[email protected]> wrote:
> Hi,
>
>
>
> This patch fixes the conflict ABI between –mabi and –target options. 
> When –mabi is specified, it now overrides the environment component of 
> the triple when conflicting is detected.
>
>
>
> The discussion is here:  http://llvm.org/bugs/show_bug.cgi?id=11326
>
>
>
> Please help the review the patch.
>
>
>
> Thanks,
>
> Weiming
>
>
>
>
>
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, 
> hosted by The Linux Foundation
>
>
>
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>

Attachment: 0002-PR-11326-Lack-diagnosic-message-when-ABI-conflicts-o.patch
Description: Binary data

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

Reply via email to