Xiangling_L created this revision. Xiangling_L added reviewers: hubert.reinterpretcast, jasonliu, daltenty. Herald added a project: clang. Herald added a subscriber: cfe-commits. Xiangling_L requested review of this revision.
Since we fixed the definition of `SuitableAlign` here: https://reviews.llvm.org/D88659, `max_align_t` and `__BIGGEST_ALIGNMENT__` should not be same always. The original testcase was added here: https://reviews.llvm.org/D59048 Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D90187 Files: clang/test/Headers/max_align.c Index: clang/test/Headers/max_align.c =================================================================== --- clang/test/Headers/max_align.c +++ /dev/null @@ -1,12 +0,0 @@ -// RUN: %clang_cc1 -fsyntax-only -std=c11 -verify %s -// expected-no-diagnostics - -// XFAIL: windows-, i686 - -#ifndef __BIGGEST_ALIGNMENT__ -#error __BIGGEST_ALIGNMENT__ not defined -#endif - -#include <stddef.h> - -_Static_assert(__BIGGEST_ALIGNMENT__ == _Alignof(max_align_t), "");
Index: clang/test/Headers/max_align.c =================================================================== --- clang/test/Headers/max_align.c +++ /dev/null @@ -1,12 +0,0 @@ -// RUN: %clang_cc1 -fsyntax-only -std=c11 -verify %s -// expected-no-diagnostics - -// XFAIL: windows-, i686 - -#ifndef __BIGGEST_ALIGNMENT__ -#error __BIGGEST_ALIGNMENT__ not defined -#endif - -#include <stddef.h> - -_Static_assert(__BIGGEST_ALIGNMENT__ == _Alignof(max_align_t), "");
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits