<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 13, 2015 at 3:10 PM Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> writes:<br>
> On Fri, Nov 13, 2015 at 2:26 PM Tim Northover <<a href="mailto:tnorthover@apple.com" target="_blank">tnorthover@apple.com</a>> wrote:<br>
>>> I think the right thing to do is to default ReserveX18 in the<br>
>>> AArch64Subtarget constructor based on isOSDarwin. i.e.:<br>
>><br>
>> I think I agree. The triple ought to be enough to give reasonably<br>
>> ABI-compliant code, otherwise why include the OS there in the first place?<br>
><br>
> Seems reasonable.<br>
<br>
I've done this in r253102.<br>
<br>
>>> If we do this we can also revert r243310 since it'll become redundant.<br>
>>> WDYT?<br>
>><br>
>> Probably a good idea too.<br>
><br>
> Sure.<br>
<br>
and this in r253104.<br>
<br>
> I think this will mean that it's not currently possible to turn it off, but<br>
> if we need that we can add a -mno- reserve option to clang.<br>
<br>
This is easy to do if someone needs it, but I don't know of anyone that<br>
does so I've left it out for now.<br></blockquote><div><br></div><div>Totally legitimate.</div><div><br></div><div>-eric</div><div> </div></div></div>