[cfe-commits] Fix PR 11326: conflicting mabi between -mabi and -target options on ARM

Weiming Zhao weimingz at codeaurora.org
Tue Mar 5 11:55:28 PST 2013


Hi Rafael,

Thanks for reviewing.
The line is needed because Opts.Triple is a string, not a Triple object.

Attached is the rebased patch (no new changes).

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:rafael.espindola at gmail.com] 
Sent: Monday, March 04, 2013 1:29 PM
To: weimingz at codeaurora.org
Cc: cfe-commits at cs.uiuc.edu; Anton Korobeynikov
Subject: Re: [cfe-commits] Fix PR 11326: conflicting mabi between -mabi and -target options on ARM

On 4 March 2013 16:20, Weiming Zhao <weimingz at codeaurora.org> wrote:
> Ping?

+  llvm::Triple T(Opts.Triple);

Can you avoid the copy with something like

const llvm::Triple &T = Opts.Triple?

OK with that change.

Cheers,
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-PR-11326-Lack-diagnosic-message-when-ABI-conflicts-o.patch
Type: application/octet-stream
Size: 4318 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130305/5f49f82b/attachment.obj>


More information about the cfe-commits mailing list