This is an archive of the discontinued LLVM Phabricator instance.

[msp430] Only support the 'm' inline assembly memory constraint. NFC.
ClosedPublic

Authored by dsanders on Mar 10 2015, 9:53 AM.

Details

Summary

MSP430 doesn't seem to have any additional constraints. Therefore remove
the target hook.

No functional change intended. Depends on D8173.

Diff Detail

Repository
rL LLVM

Event Timeline

dsanders updated this revision to Diff 21596.Mar 10 2015, 9:53 AM
dsanders retitled this revision from to [msp430] Only support the 'm' inline assembly memory constraint. NFC..
dsanders updated this object.
dsanders edited the test plan for this revision. (Show Details)
dsanders added a subscriber: Unknown Object (MLST).

Ping.

The main thing I'm looking for is for is for someone with knowledge of the target to say whether they're aware of any supported constraints that this patch doesn't cover.

Ping 2...

I'm not sure that there is active development on MSP430 so I'm not sure who would review this. Looking through the commit logs for likely reviewers I can only see infrastructure changes this year..

I had a quick chat with Eric Christopher at EuroLLVM about this patch and he suggested I go ahead and commit this one.

This revision was automatically updated to reflect the committed changes.