Page MenuHomePhabricator

[AArch64] Add default feature for CPUs on AArch64 target in Clang.
Needs ReviewPublic

Authored by kevin.qin on Jun 10 2014, 1:01 AM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

For ARM target, we can use CRYPTO and CRC features if we select cortex-a57 by '-mcpu', but for AArch64 target, it doesn't work unless adding with '-mfpu=crypto-neon-fp-armv8'. The reason is, default feature for AArch64 target CPUs isn't set in front-end. To keep consistency between front-end and back-end and get end-users more easier to use, we'd better add default feature for CPUs on AArch64 target as well.

Diff Detail

Event Timeline

kevin.qin updated this revision to Diff 10264.Jun 10 2014, 1:01 AM
kevin.qin retitled this revision from to [AArch64] Add default feature for CPUs on AArch64 target in Clang..
kevin.qin updated this object.
kevin.qin edited the test plan for this revision. (Show Details)
kevin.qin added subscribers: Unknown Object (MLST), Unknown Object (MLST).

Hi Kevin,

This looks sensible to me.

Cheers.

Tim.