Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[clang] Disable gch-probe.c on AIX as -gmodules is not supported there yet. #78513

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

Bigcheese
Copy link
Contributor

Followup fix for #77711

@llvmbot llvmbot added the clang Clang issues not falling into any other category label Jan 17, 2024
@llvmbot
Copy link
Collaborator

llvmbot commented Jan 17, 2024

@llvm/pr-subscribers-clang

Author: Michael Spencer (Bigcheese)

Changes

Followup fix for #77711


Full diff: https://github.com/llvm/llvm-project/pull/78513.diff

1 Files Affected:

  • (modified) clang/test/PCH/gch-probe.c (+4)
diff --git a/clang/test/PCH/gch-probe.c b/clang/test/PCH/gch-probe.c
index 0905c9baebdae0..370abd26d25174 100644
--- a/clang/test/PCH/gch-probe.c
+++ b/clang/test/PCH/gch-probe.c
@@ -1,3 +1,7 @@
+// Unsupported on AIX because we don't support the requisite "__clangast"
+// section in XCOFF yet.
+// UNSUPPORTED: target={{.*}}-aix{{.*}}
+
 // For GCC compatibility, clang should probe also with the .gch extension.
 // RUN: %clang -x c-header -c %s -o %t.h.gch
 // RUN: %clang -fsyntax-only -include %t.h %s

@Bigcheese Bigcheese merged commit 0f67c7a into llvm:main Jan 17, 2024
4 of 5 checks passed
@Bigcheese Bigcheese deleted the dev/fix-pch-aix branch January 17, 2024 22:40
ampandey-1995 pushed a commit to ampandey-1995/llvm-project that referenced this pull request Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants