[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