This is an archive of the discontinued LLVM Phabricator instance.

[MSP430] Add subtarget features for hardware multiplier.
ClosedPublic

Authored by pftbest on May 18 2017, 1:16 PM.

Details

Event Timeline

pftbest created this revision.May 18 2017, 1:16 PM
awygle edited edge metadata.May 21 2017, 6:40 PM

This looks good to me, I'd just like a test for the -mhwmult command line option as well as the -mattr version.

I'd just like a test for the -mhwmult command line option as well as the -mattr version.

But they are both present. For example hwmult16.ll file has two RUN commands one for -mhwmult and other for -mattr

awygle accepted this revision.May 23 2017, 11:43 AM

I completely misread that, sorry - just assumed the line above was removed despite the total lack of red.

LGTM.

This revision is now accepted and ready to land.May 23 2017, 11:43 AM
This revision was automatically updated to reflect the committed changes.