[PATCH] D82668: [AArch64][SVE] clang: Add missing svbfloat16_t tests

[Francesco Petrogalli via Phabricator via cfe-commits]

fpetrogalli added inline comments.


================
Comment at: clang/test/CodeGenCXX/aarch64-sve-typeinfo.cpp:4
 // RUN: %clang_cc1 -triple aarch64-none-linux-gnu %s -emit-llvm -o - \
-// RUN:   -target-feature +sve | FileCheck %s
+// RUN:   -target-feature +sve,+bf16 | FileCheck %s
 
----------------
I wonder if we should keep the bf16 tests separate, as +bf16 is not needed to generate any of the other SVE types. I don't have strong opinions here, but it seems a better thing to do to isolate the bfloat tests in SVE in a separate file. What do you think, @c-rhodes?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82668