[libcxx-commits] [PATCH] D144364: [libcxx] <experimental/simd> Added internal storage type, constructors, subscript operators of class simd/simd_mask and related tests

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added a comment.

In D144364#4636494 <https://reviews.llvm.org/D144364#4636494>, @Joy12138 wrote:

> It seems that the `long double` is a type with a size of 12 bytes on `MinGW (DLL, i686)`. This conflicts with the current implementation. Vector extension requires the input parameter of vector size to be a power of 2 and a multiple of the size of the element type. 12 bytes cannot meet both requirements simultaneously. Should we temporarily exclude `long double` on `MinGW (DLL, i686)`? I think it would be more appropriate to add support for it when implement platform specialization optimization. @philnik

Sounds fine to me with a TODO.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144364