[PATCH] D65404: [AArch64] Disable __ARM_FEATURE_SVE without ACLE.

[Renato Golin via Phabricator via cfe-commits]

rengolin added a comment.

I understand (and agree with) the reasoning of this patch, but wouldn't this also make it harder to test the current behaviour?

I mean, LLVM doesn't support officially SVE entirely, so there's no point in expecting anything to work for now. What is the coverage of the current ACLE implementation in Clang/LLVM?

If we already have the most used ones, how about adding a warning that SVE support is not complete, YMMV, etc. instead?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65404/new/

https://reviews.llvm.org/D65404