[libcxx-commits] [PATCH] D46443: [libc++] Add missing cstdalign header
Sam Clegg via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Sat Mar 6 18:46:38 PST 2021
sbc100 added inline comments.
Comment at: libcxx/include/cstdalign:24
> This seems to be assuming that the underlying C library's `stdalign.h` is C++ friendly. A C11 `stdalign.h` //does// define `alignof` and `alignas` as macros.
Should I just remove this `#include` then?
Comment at: libcxx/test/std/language.support/cstdalign/cstdalign.pass.cpp:27
+#error __alignof_is_defined not defined
> sbc100 wrote:
> > ldionne wrote:
> > > I'm not seeing `__alignof_is_defined` anywhere in the spec?
> > Removed
> Seems like a defect in the old standard. The prose doesn't match the synopsis. `__alignof_is_defined` is a macro in C11's `stdalign.h` (and so is `alignof`). That the C++ committee did not intend for an `alignof` macro can probably be assumed. I suspect the lack of an `__alignof_is_defined` macro was also unintended.
So should I add back the check for `__alignof_is_defined`?
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the libcxx-commits