[libcxx-commits] [PATCH] D144363: [libcxx] <experimental/simd> Added simd width functions, simd_size traits and related tests

Wed Sep 13 10:37:08 PDT 2023

mgorny added a comment.

In D144363#4644854 <https://reviews.llvm.org/D144363#4644854>, @Joy12138 wrote:

> Thank you very much for your reminder. I tried to fix this issue in revision https://reviews.llvm.org/D159509 and land this patch after CI passed.

I'm getting the same errors after this change (tested on 2f005df066e07d93e3d6aa04748c158f883197b7 <https://reviews.llvm.org/rG2f005df066e07d93e3d6aa04748c158f883197b7>).

Note the specific errors are:

  /var/tmp/portage/sys-libs/libcxx-18.0.0_pre20230912/work/runtimes_build-abi_x86_32.x86/include/c++/v1/experimental/__simd/vec_ext.h:31:29: error: vector size not an integral multiple of component size
     31 |   _Tp __data __attribute__((__vector_size__(std::__bit_ceil((sizeof(_Tp) * _Np)))));
        |                             ^               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

and:

  /var/tmp/portage/sys-libs/libcxx-18.0.0_pre20230912/work/runtimes_build-abi_x86_32.x86/include/c++/v1/experimental/__simd/utility.h:53:19: error: static assertion failed due to requirement 'sizeof(long double) == 0': Unexpected size
     53 |     static_assert(sizeof(_Tp) == 0, "Unexpected size");
        |                   ^~~~~~~~~~~~~~~~

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144363