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

Thu Sep 14 04:24:59 PDT 2023

Joy12138 added a comment.

In D144363#4645309 <https://reviews.llvm.org/D144363#4645309>, @mgorny wrote:

> 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");
>         |                   ^~~~~~~~~~~~~~~~
>
>   

Sorry. Because this issue did not appear in the CI of above revision, I presumed that it is related to another issue that appeared in the CI. But it does not seem like this. I am trying to reproduce and resolve it by https://reviews.llvm.org/D159514. This time, I will ensure that the modifications can pass the test on 2f005df066e07d93e3d6aa04748c158f883197b7 <https://reviews.llvm.org/rG2f005df066e07d93e3d6aa04748c158f883197b7>

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144363