[PATCH] D118670: [NFC][AIX]Disable failed tests due to aggressive byval alignment warning on AIX

Steven Wan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 31 17:29:27 PST 2022


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


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118670.404772.patch
Type: text/x-patch
Size: 2061 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220201/17677ae4/attachment.bin>


More information about the cfe-commits mailing list