[PATCH] D82178: [AArch64][SVE] Guard svbfloat16_t with feature macro in ACLE

Francesco Petrogalli via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 19 07:32:46 PDT 2020


fpetrogalli added inline comments.


================
Comment at: clang/utils/TableGen/SveEmitter.cpp:1092
+  OS << "#if !defined(__ARM_FEATURE_BF16_SCALAR_ARITHMETIC)\n";
+  OS << "#error \"__ARM_FEATURE_BF16_SCALAR_ARITHMETIC must be defined when "
+        "__ARM_FEATURE_SVE_BF16 is defined\"\n";
----------------
Does it make sense to add a regression test to make sure this error is raised?


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

https://reviews.llvm.org/D82178





More information about the cfe-commits mailing list