[libcxx-commits] [PATCH] D46443: Add missing cstdalign header

Sam Clegg via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Mar 5 09:40:14 PST 2021


sbc100 marked 2 inline comments as done.
sbc100 added inline comments.


================
Comment at: libcxx/test/std/language.support/cstdalign/cstdalign.pass.cpp:14-28
+#ifdef alignas
+#error alignas should not be a macro in C++
+#endif
+
+#ifdef alignof
+#error alignof should not be a macro in C++
+#endif
----------------
ldionne wrote:
> This can be moved outside of `main()`.
I was copying from `libcxx/test/std/language.support/cstdint/cstdint.syn/cstdint.pass.cpp` which does these checks inside of main.   

Will move them.


================
Comment at: libcxx/test/std/language.support/cstdalign/cstdalign.pass.cpp:27
+#ifndef __alignof_is_defined
+#error __alignof_is_defined not defined
+#endif
----------------
ldionne wrote:
> I'm not seeing `__alignof_is_defined` anywhere in the spec?
Removed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D46443



More information about the libcxx-commits mailing list