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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to