This is an archive of the discontinued LLVM Phabricator instance.

Add more ChangeTo* functions to MachineOperand
ClosedPublic

Authored by arsenm on May 5 2015, 3:17 PM.

Details

Reviewers
hfinkel
Summary

Allow replacing with a symbol

Diff Detail

Event Timeline

arsenm updated this revision to Diff 24987.May 5 2015, 3:17 PM
arsenm retitled this revision from to Add more ChangeTo* functions to MachineOperand.
arsenm updated this object.
arsenm edited the test plan for this revision. (Show Details)
arsenm added a subscriber: Unknown Object (MLST).
hfinkel added a subscriber: hfinkel.May 5 2015, 7:24 PM
hfinkel added inline comments.
include/llvm/CodeGen/MachineOperand.h
558

Please write out ChangeToExternalSymbol

arsenm added inline comments.May 5 2015, 8:53 PM
include/llvm/CodeGen/MachineOperand.h
558

This was to match the name of CreateES

hfinkel accepted this revision.May 5 2015, 11:56 PM
hfinkel added a reviewer: hfinkel.

LGTM.

include/llvm/CodeGen/MachineOperand.h
558

Alright (that's unfortunate ;) ).

This revision is now accepted and ready to land.May 5 2015, 11:56 PM
arsenm closed this revision.May 6 2015, 10:09 AM

r236612