<br><br><div class="gmail_quote">On Wed Dec 10 2014 at 7:31:37 AM Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 9 December 2014 at 21:55, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
> Ping? Do you have a reason for this change? Basically it means a lot of<br>
> updating the back end if we want them to agree for no good reason so I'm of<br>
> mind to revert this.<br>
<br>
Hi Eric,<br>
<br>
Is this causing any trouble?<br></blockquote><div><br></div><div>So a bit. I'm trying to unify the code and as I said earlier, it makes the default ABI different between the front end and backend.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
IIUC, this change will only have an effect when no environment is<br>
chosen, which shouldn't happen that often, and shouldn't happen at all<br>
on Darwin or BSD. The code as it currently is has even an extra<br>
condition for NetBSD reverting to "apcs-gnu".<br>
<br>
Since these things are largely arbitrary, creating a code that makes<br>
sense and is simple is hard. But regarding the decision to move the<br>
default to aapcs, I have to say I also agree.<br></blockquote><div><br></div><div>Agree with what though? For bare metal people should be using aapcs? Yeah, I agree. For that we should just define an ABI for arm-elf and do that. That said, the default is different between the front end and back end is different for no reason now and fixing the tests in the back end is much more complicated than just reverting this. As you said earlier, every other case just explicitly is recognized.</div><div><br></div><div>Since it doesn't seem like there's a reason other than "it seemed like a nice idea" (which I agree with, just that it's more work than just the front end change) I'm going to revert this for now and make sure the ABI definitions are unified between the front and back end. If we decide we really want to change it then it can be done in both the front end and back end with the requisite testsuite updates.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
cheers,<br>
--renato<br>
</blockquote></div>