[PATCH] D53207: Fix bug 26547 - alignof should return ABI alignment, not preferred alignment
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 12 18:21:02 PDT 2018
rsmith added a comment.
Please extend the tests to check that the `-fclang-abi-compat=7` switch undoes your change here.
One other comment, and otherwise this looks good. Will you need someone to commit this for you?
================
Comment at: test/SemaCXX/align-x86.cpp:4-5
+
+#include <complex>
+#include <cstddef>
+
----------------
It's not OK to include standard library headers here -- there's no guarantee that anything sensible will be on the include path, and we want our tests to be self-contained. (Feel free to define your own mini-`std::complex` in this test.)
Repository:
rC Clang
https://reviews.llvm.org/D53207
More information about the cfe-commits
mailing list