r216662 - [ARM] Change default ABI for AArch32 to be "aapcs" (was "apcs-gnu")

Renato Golin renato.golin at linaro.org
Wed Dec 10 12:28:48 PST 2014


On 10 December 2014 at 19:46, Eric Christopher <echristo at gmail.com> wrote:
> 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.

That's a good point. The code as it is doesn't seem like it was well
thought, but hacked in by different people at different times and
"approved" by people that didn't care much what the default is (like
myself).

Reverting is probably the safest thing to do, right now. If that
creates problems for Oliver, than we should get more concrete reasons
and plans than just a "nice idea".

cheers,
--renato



More information about the cfe-commits mailing list