stevewan created this revision.
stevewan requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
These tests emit unexpected diagnostics on AIX because the byval alignment
warning is emitted too aggressively. https://reviews.llvm.org/D118350 is
supposed to provide a proper fix to the problem, but for the time being disable
the tests to unblock.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D118670
Files:
clang/test/Analysis/padding_c.c
clang/test/Analysis/padding_cpp.cpp
clang/test/CXX/drs/dr6xx.cpp
clang/test/SemaTemplate/instantiate-attr.cpp
Index: clang/test/SemaTemplate/instantiate-attr.cpp
===================================================================
--- clang/test/SemaTemplate/instantiate-attr.cpp
+++ clang/test/SemaTemplate/instantiate-attr.cpp
@@ -1,5 +1,9 @@
// RUN: %clang_cc1 -fsyntax-only -verify %s
// expected-no-diagnostics
+
+// Byval alignment warning is emitted too aggressively on AIX.
+// XFAIL: aix
+
template <typename T>
struct A {
char a __attribute__((aligned(16)));
Index: clang/test/CXX/drs/dr6xx.cpp
===================================================================
--- clang/test/CXX/drs/dr6xx.cpp
+++ clang/test/CXX/drs/dr6xx.cpp
@@ -4,6 +4,9 @@
// RUN: %clang_cc1 -std=c++17 %s -verify -fexceptions -fcxx-exceptions
-pedantic-errors -fno-spell-checking
// RUN: %clang_cc1 -std=c++20 %s -verify -fexceptions -fcxx-exceptions
-pedantic-errors -fno-spell-checking
+// Byval alignment warning is emitted too aggressively on AIX.
+// XFAIL: aix
+
namespace std {
struct type_info {};
__extension__ typedef __SIZE_TYPE__ size_t;
Index: clang/test/Analysis/padding_cpp.cpp
===================================================================
--- clang/test/Analysis/padding_cpp.cpp
+++ clang/test/Analysis/padding_cpp.cpp
@@ -1,5 +1,8 @@
// RUN: %clang_analyze_cc1 -std=c++14 -analyzer-checker=optin.performance
-analyzer-config optin.performance.Padding:AllowedPad=2 -verify %s
+// Byval alignment warning is emitted too aggressively on AIX.
+// XFAIL: aix
+
// Make sure that the C cases still work fine, even when compiled as C++.
#include "padding_c.c"
Index: clang/test/Analysis/padding_c.c
===================================================================
--- clang/test/Analysis/padding_c.c
+++ clang/test/Analysis/padding_c.c
@@ -12,6 +12,9 @@
// CHECK-PAD-NEGATIVE-VALUE-SAME: 'optin.performance.Padding:AllowedPad', that
// CHECK-PAD-NEGATIVE-VALUE-SAME: expects a non-negative value
+// Byval alignment warning is emitted too aggressively on AIX.
+// XFAIL: aix
+
#if __has_include(<stdalign.h>)
#include <stdalign.h>
#endif
Index: clang/test/SemaTemplate/instantiate-attr.cpp
===================================================================
--- clang/test/SemaTemplate/instantiate-attr.cpp
+++ clang/test/SemaTemplate/instantiate-attr.cpp
@@ -1,5 +1,9 @@
// RUN: %clang_cc1 -fsyntax-only -verify %s
// expected-no-diagnostics
+
+// Byval alignment warning is emitted too aggressively on AIX.
+// XFAIL: aix
+
template <typename T>
struct A {
char a __attribute__((aligned(16)));
Index: clang/test/CXX/drs/dr6xx.cpp
===================================================================
--- clang/test/CXX/drs/dr6xx.cpp
+++ clang/test/CXX/drs/dr6xx.cpp
@@ -4,6 +4,9 @@
// RUN: %clang_cc1 -std=c++17 %s -verify -fexceptions -fcxx-exceptions -pedantic-errors -fno-spell-checking
// RUN: %clang_cc1 -std=c++20 %s -verify -fexceptions -fcxx-exceptions -pedantic-errors -fno-spell-checking
+// Byval alignment warning is emitted too aggressively on AIX.
+// XFAIL: aix
+
namespace std {
struct type_info {};
__extension__ typedef __SIZE_TYPE__ size_t;
Index: clang/test/Analysis/padding_cpp.cpp
===================================================================
--- clang/test/Analysis/padding_cpp.cpp
+++ clang/test/Analysis/padding_cpp.cpp
@@ -1,5 +1,8 @@
// RUN: %clang_analyze_cc1 -std=c++14 -analyzer-checker=optin.performance -analyzer-config optin.performance.Padding:AllowedPad=2 -verify %s
+// Byval alignment warning is emitted too aggressively on AIX.
+// XFAIL: aix
+
// Make sure that the C cases still work fine, even when compiled as C++.
#include "padding_c.c"
Index: clang/test/Analysis/padding_c.c
===================================================================
--- clang/test/Analysis/padding_c.c
+++ clang/test/Analysis/padding_c.c
@@ -12,6 +12,9 @@
// CHECK-PAD-NEGATIVE-VALUE-SAME: 'optin.performance.Padding:AllowedPad', that
// CHECK-PAD-NEGATIVE-VALUE-SAME: expects a non-negative value
+// Byval alignment warning is emitted too aggressively on AIX.
+// XFAIL: aix
+
#if __has_include(<stdalign.h>)
#include <stdalign.h>
#endif
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
