This is an archive of the discontinued LLVM Phabricator instance.

The operand flag word used in ISD::INLINEASM is an i32 not a pointer. NFC.
ClosedPublic

Authored by dsanders on Mar 9 2015, 6:55 AM.

Details

Summary

This is part of the work to support memory constraints that behave
differently to 'm'. The subsequent patches will expand on the existing
encoding (which is a 32-bit int) and as a result in some flag words will no
longer fit into an i16. This problem only affected the MSP430 target which
appears to have 16-bit pointers.

Diff Detail

Event Timeline

dsanders updated this revision to Diff 21486.Mar 9 2015, 6:55 AM
dsanders retitled this revision from to The operand flag word used in ISD::INLINEASM is an i32 not a pointer. NFC..
dsanders updated this object.
dsanders edited the test plan for this revision. (Show Details)
dsanders added a subscriber: Unknown Object (MLST).
hfinkel accepted this revision.Mar 9 2015, 2:01 PM
hfinkel added a reviewer: hfinkel.
hfinkel added a subscriber: hfinkel.

LGTM.

This revision is now accepted and ready to land.Mar 9 2015, 2:01 PM
dsanders closed this revision.Mar 10 2015, 3:45 AM