<html dir="ltr">

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style id="owaParaStyle" type="text/css">P {margin-top:0;margin-bottom:0;}</style>

</head>

<body ocsi="0" fpstyle="1">

<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">> So now you have a situation where there are two methods, that are both happening at the same time, of passing the ABI to the backend. There needs to be one.

<br>

<br>

<font size="2">That's not caused by this patch (or series), it's been the case since r224492<span lang="en-GB"><font face="Times New Roman,serif" size="3"><span style="font-size:12pt;"><font face="Calibri,sans-serif" size="2"><span style="font-size:11pt;"><span lang="en-GB"><font face="Times New Roman,serif" size="3"><span style="font-size:12pt;"><font face="Calibri,sans-serif" size="2"><span style="font-size:11pt;">.

 This patch just makes it obvious because a command line option is doing the substitution instead of the human writing the tests.<br>

<br>

There are three camps today:<br>

* ARM/PowerPC pass the ABI in _both_ the triple and MCTargetOptions<br>

* Mips passes the ABI in the triple in some cases (O32, N64) and MCTargetOptions in other cases (N32).<br>

* X86/everyone-else pass the ABI in the triple only<br>

<br>

My patch series started out trying to reduce this to:<br>

* ARM/PowerPC pass the ABI in _both_ the triple and MCTargetOptions<br>

* Mips/X86/everyone-else pass the ABI in the triple only<br>

<br>

I'm currently trying to re-work the plan to reduce this further one of these three options:<br>

* ARM/PowerPC/Mips/X86/everyone-else pass the ABI in _both_ the triple and MCTargetOptions<br>

<strike>* ARM/PowerPC/Mips/X86/everyone-else pass the ABI in MCTargetOptions only</strike> (already ruled out because MCTargetOptions is not sufficient for ARM/X86/Mips)<br>

* ARM/PowerPC/Mips/X86/everyone-else pass the ABI in the triple only. (not ruled out yet but is significantly trickier than the first option)<br>

The comments you're replying to are in line with aiming for the first of these three options.<br>

<br>

I can't reach the end point in a single leap without breaking things or ending up with a truly massive patch. I need you to let me incrementally move towards it so that the trunk always works.<br>

<br>

<div style="font-family: Times New Roman; color: #000000; font-size: 16px">

<hr tabindex="-1">

<div style="direction: ltr;" id="divRpF337022"><font face="Tahoma" color="#000000" size="2"><b>From:</b> Eric Christopher [echristo@gmail.com]<br>

<b>Sent:</b> 11 July 2016 18:18<br>

<b>To:</b> reviews+D21467+public+827b718e7f297099@reviews.llvm.org; Daniel Sanders<br>

<b>Cc:</b> llvm-commits@lists.llvm.org<br>

<b>Subject:</b> Re: [PATCH] D21467: [llc+llvm-mc] Prefer applying -mabi to the triple where possible and allow Mips to take ABI information in the triple.<br>

</font><br>

</div>

<div></div>

<div>

<div dir="ltr">So now you have a situation where there are two methods, that are both happening at the same time, of passing the ABI to the backend. There needs to be one.

<div><br>

</div>

<div><br>

</div>

</div>

<br>

<div class="gmail_quote">

<div dir="ltr">On Mon, Jul 11, 2016 at 8:52 AM Daniel Sanders via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>

</div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">

dsanders added a comment.<br>

<br>

> This patch only moves the Mips target to this mechanism at the moment. I'll develop the ARM/X86/other<br>

<br>

>  patches separately and we can either commit them in sequence or fold them all into a single patch.<br>

<br>

<br>

X86 doesn't need any changes because it doesn't support -target-abi (or -mabi in clang) at all.<br>

<br>

PowerPC doesn't need any changes because the ABI only affects codegen.<br>

<br>

That just leaves ARM since no other target is using -target-abi. I'm still confirming whether we need any changes to account for ARM.<br>

<br>

<br>

<a href="http://reviews.llvm.org/D21467" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21467</a><br>

<br>

<br>

<br>

_______________________________________________<br>

llvm-commits mailing list<br>

<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>

<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>

</blockquote>

</div>

</div>

</div>

</div>

</body>

</html>