[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