<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 13, 2015 at 2:26 PM Tim Northover <<a href="mailto:tnorthover@apple.com">tnorthover@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> 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 ABI-compliant code, otherwise why include the OS there in the first place?<br></blockquote><div><br></div><div>Seems reasonable.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> If we do this we can also revert r243310 since it'll become redundant. WDYT?<br>
<br>
Probably a good idea too.<br></blockquote><div><br></div><div>Sure.</div><div><br></div><div>I think this will mean that it's not currently possible to turn it off, but if we need that we can add a -mno- reserve option to clang.</div><div><br></div><div>-eric </div></div></div>