[PATCH] D82178: [AArch64][SVE] Guard svbfloat16_t with feature macro in ACLE
    Francesco Petrogalli via Phabricator via llvm-commits 
    llvm-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 llvm-commits
mailing list