Skip to content

Conversation

@Jason-VanBeusekom
Copy link
Contributor

@Jason-VanBeusekom Jason-VanBeusekom commented Dec 2, 2025

4394aa6 introduced the test amdgcn_weak_alias, which is failing on the reverse iteration build, due to the the order of the aliasees being different. This failure is a test issue, not a bug, so the metadata checks are removed.

db686dc introduced the test amdgcn_weak_alias,
which is failing on the reverse iteration build, due to the
the order of the aliasees being different. This failure is a test issue,
not a bug, so the metadata checks are removed.
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:openmp OpenMP related changes to Clang labels Dec 2, 2025
@llvmbot
Copy link
Member

llvmbot commented Dec 2, 2025

@llvm/pr-subscribers-clang

Author: None (Jason-VanBeusekom)

Changes

db686dc introduced the test amdgcn_weak_alias, which is failing on the reverse iteration build, due to the the order of the aliasees being different. This failure is a test issue, not a bug, so the metadata checks are removed.


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

1 Files Affected:

  • (modified) clang/test/OpenMP/amdgcn_weak_alias.c (-7)
diff --git a/clang/test/OpenMP/amdgcn_weak_alias.c b/clang/test/OpenMP/amdgcn_weak_alias.c
index a9d5c1737b321..33c7dc0041810 100644
--- a/clang/test/OpenMP/amdgcn_weak_alias.c
+++ b/clang/test/OpenMP/amdgcn_weak_alias.c
@@ -94,10 +94,3 @@ int Three(void) __attribute__ ((weak, alias("__Three")));
 int Three_(void) __attribute__ ((alias("__Three")));
 extern int __attribute__((weak, alias("__Three_var"))) Three_var;
 extern int __attribute__((alias("__Three_var"))) Three_var_;
-//.
-// HOST: [[META0:![0-9]+]] = !{i32 1, !"__Two_var", i32 0, i32 0}
-// HOST: [[META1:![0-9]+]] = !{i32 1, !"__Three_var", i32 0, i32 1}
-//.
-// DEVICE: [[META0:![0-9]+]] = !{i32 1, !"__Two_var", i32 0, i32 0}
-// DEVICE: [[META1:![0-9]+]] = !{i32 1, !"__Three_var", i32 0, i32 1}
-//.

@Jason-VanBeusekom
Copy link
Contributor Author

@jhuber6 @yxsamliu

@jhuber6 jhuber6 enabled auto-merge (squash) December 2, 2025 16:56
@jhuber6 jhuber6 merged commit 9063416 into llvm:main Dec 2, 2025
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:openmp OpenMP related changes to Clang clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants