efriedma added inline comments.

================
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:1878
+
+    llvm_unreachable("Unexpected initialization of a scalable vector!");
+  }
----------------
paulwalker-arm wrote:
> efriedma wrote:
> > I can see why you can't have more than one element... but both zero and one 
> > seem feasible.  (For example, `__SVInt8_t s8{__SVInt8_t{}};`.)
> I thought the same but anything more that zero triggers a `cannot initialize 
> a value of type '__SVFloat32_t' with an rvalue of type...` compilation error 
> message.  Since I got the same error with gcc I didn't dig any deeper.
Let me put it this way: the following crashes on main, and this patch doesn't 
address it.

```
void test_locals(__SVInt8_t x) {
  __SVInt8_t s8{x};
}
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153560

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to