This is an archive of the discontinued LLVM Phabricator instance.

MIPS now uses 'or' for move instead of [d]addu.
ClosedPublic

Authored by sdardis on Aug 18 2015, 7:11 AM.

Details

Diff Detail

Event Timeline

sdardis updated this revision to Diff 32411.Aug 18 2015, 7:11 AM
sdardis retitled this revision from to MIPS now uses 'or' for move instead of [d]addu..
sdardis updated this object.
sdardis added a reviewer: atanasyan.
sdardis added a project: lld.
sdardis added a subscriber: llvm-commits.
atanasyan accepted this revision.Aug 18 2015, 9:08 AM
atanasyan edited edge metadata.

LGTM But there are three additional test cases require update: la25-stub-npic-shared.test, plt-entry-mixed-2.test, plt-entry-mixed-4.test. Do you have commit access? If you do not, I can commit the patch with the fix for three additional test cases.

This revision is now accepted and ready to land.Aug 18 2015, 9:08 AM
sdardis updated this revision to Diff 32527.Aug 19 2015, 3:04 AM
sdardis edited edge metadata.

Updated la25-stub-npic-shared.test, plt-entry-mixed-2.test, plt-entry-mixed-4.test. I don't have commit access.

Thanks.

This revision was automatically updated to reflect the committed changes.

Committed at r245491. Thanks for the patch.