This is an archive of the discontinued LLVM Phabricator instance.

[mips] Clean up the CodeGen/Mips/inlineasmmemop.ll test. NFC.
ClosedPublic

Authored by tomatabacu on Dec 17 2014, 7:28 AM.

Details

Summary

Improve comments and remove a redundant attribute list.
There are no functional changes (to the CHECK's or to the code).

Part of these changes were suggested in http://reviews.llvm.org/D6637.

Diff Detail

Event Timeline

tomatabacu updated this revision to Diff 17393.Dec 17 2014, 7:28 AM
tomatabacu retitled this revision from to [mips] Clean up the CodeGen/Mips/inlineasmmemop.ll test. NFC..
tomatabacu updated this object.
tomatabacu edited the test plan for this revision. (Show Details)
tomatabacu added a reviewer: dsanders.
tomatabacu added a subscriber: Unknown Object (MLST).
dsanders accepted this revision.Dec 18 2014, 3:48 AM
dsanders edited edge metadata.

LGTM. You ought to have made these changes before committing r224425.

This revision is now accepted and ready to land.Dec 18 2014, 3:48 AM
tomatabacu closed this revision.Dec 18 2014, 5:04 AM