[libcxx-commits] [PATCH] D156225: [libcxx] <experimental/simd> Add broadcast constructor of class simd/simd_mask

Fri Aug 11 08:03:07 PDT 2023

philnik added a comment.

In D156225#4579302 <https://reviews.llvm.org/D156225#4579302>, @Joy12138 wrote:

> The CI errors appear to be related to the type `__int128` and `__int128  unsigned`. I tried these types and it is not supported in GCC implementation (https://godbolt.org/z/EExodWKxE), too. Should we support them? Or where should they be excluded? @philnik

I don't really see a reason not to support it. A simple test seems to suggest that `__int128` works just fine with vector types: https://godbolt.org/z/W9v6T6c8d. So my suspicion is that there is a bug somewhere in your code. It could of course also be a compiler bug, which is supported by the fact that Clang is happy with the code while GCC isn't. Either way, I think we should investigate this.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156225