[libcxx-commits] [libcxx] [libc++][string] Fix unnecessary padding for basic_string's __short (PR #135973)

[Gábor Spaits via libcxx-commits]

spaits wrote:

@philnik777 The build failed on my downstream, target with exactly the same reason as the current CI:
https://buildkite.com/llvm-project/libcxx-ci/builds/42032#01963f22-4987-4bec-87d9-413732948d0a AIX (32 bit) target.

When I reverted the change d95597dc06c510ad7fbf00a43583c54d38f79aa7 the assertion failure has disappeared. Also I looked up how empty structs are handled and then I saw, that they can not be empty on C++, so I assumed, the reason for the failure was that change.

https://github.com/llvm/llvm-project/pull/135973