This is an archive of the discontinued LLVM Phabricator instance.

[mips] Allow ABI names from MCTargetOptions::ABIName but only if they agree with the triple.
Needs ReviewPublic

Authored by sdardis on Jul 13 2016, 8:23 AM.

Details

Summary

This brings the last of the targets into the 'pass in both the triple and
MCTargetOptions' camp.

Depends on D21069. I had expected to need a corresponding clang patch but
this proved unnecessary.

Patch by: Daniel Sanders

Diff Detail

Event Timeline

dsanders updated this revision to Diff 63807.Jul 13 2016, 8:23 AM
dsanders retitled this revision from to [mips] Allow ABI names from MCTargetOptions::ABIName but only if they agree with the triple..
dsanders updated this object.
dsanders updated this revision to Diff 64882.Jul 21 2016, 7:12 AM

Refresh and ping.

rafael removed a subscriber: rafael.
dsanders updated this revision to Diff 65925.Jul 28 2016, 6:44 AM

Refresh and ping

One last ping since I need to either commit this series next week or hand over to a colleague to continue it.

sdardis commandeered this revision.Aug 23 2016, 6:26 AM
sdardis added a reviewer: dsanders.

Taking over this patch series.

sdardis updated this revision to Diff 68983.Aug 23 2016, 6:29 AM
sdardis updated this object.
sdardis edited edge metadata.

Rebase + ping. I am aware that you're exploring your alternate solution Eric.

sdardis updated this revision to Diff 76727.Nov 2 2016, 10:01 AM

Rebase and ping.

dsanders resigned from this revision.Jul 18 2019, 6:49 PM