[PATCH] D78831: [SVE] Upgrade VectorType tests to test new types

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 2 03:19:16 PDT 2020


fhahn added inline comments.


================
Comment at: llvm/unittests/IR/VectorTypesTest.cpp:231
+
+  std::array<VectorType *, 8> VTys = {VectorType::get(Int16Ty, {4, true}),
+                                      VectorType::get(Int16Ty, {4, false}),
----------------
This seems to cause the following warning on GreenDragon (
http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-expensive/15985/console)

It would be great if you could take a look.

```
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-expensive/llvm-project/llvm/unittests/IR/VectorTypesTest.cpp:231:39: warning: suggest braces around initialization of subobject [-Wmissing-braces]
  std::array<VectorType *, 8> VTys = {VectorType::get(Int16Ty, {4, true}),
                                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                      {
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78831





More information about the llvm-commits mailing list