This is an archive of the discontinued LLVM Phabricator instance.

Add support for armv7ve flag in clang (PR31358).
ClosedPublic

Authored by manojgupta on Feb 9 2017, 10:16 AM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

manojgupta created this revision.Feb 9 2017, 10:16 AM

This all LGTM. If I can assume I am allow to approve, then I approve.

test/Preprocessor/arm-acle-6.4.c
136 ↗(On Diff #87831)

Seems like we could check for __ARM_FEATURE_IDIV here too since we know it must be there.

Oh - I see it is tested in another test, in which case fine.

I guess these test files are a bit of a mess already.

This revision is now accepted and ready to land.Feb 9 2017, 11:50 AM
This revision was automatically updated to reflect the committed changes.