Skip to content

Conversation

@lntue
Copy link
Contributor

@lntue lntue commented Mar 22, 2025

This test causes clang to crash due to #132494.

@lntue lntue requested a review from jhuber6 March 22, 2025 01:54
@llvmbot llvmbot added the libc label Mar 22, 2025
@llvmbot
Copy link
Member

llvmbot commented Mar 22, 2025

@llvm/pr-subscribers-libc

Author: None (lntue)

Changes

This test causes clang to crash due to #132494.


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

1 Files Affected:

  • (modified) libc/test/src/__support/CPP/type_traits_test.cpp (+3-1)
diff --git a/libc/test/src/__support/CPP/type_traits_test.cpp b/libc/test/src/__support/CPP/type_traits_test.cpp
index 4b3e48c6a6c0f..85e71f9d90026 100644
--- a/libc/test/src/__support/CPP/type_traits_test.cpp
+++ b/libc/test/src/__support/CPP/type_traits_test.cpp
@@ -334,7 +334,9 @@ TEST(LlvmLibcTypeTraitsTest, is_class) {
   // Neither other types.
   EXPECT_FALSE((is_class_v<Union>));
   EXPECT_FALSE((is_class_v<int>));
-  EXPECT_FALSE((is_class_v<EnumClass>));
+  // TODO: Re-enable the test after
+  // https://github.com/llvm/llvm-project/issues/132494 is fixed.
+  // EXPECT_FALSE((is_class_v<EnumClass>));
 }
 
 TYPED_TEST(LlvmLibcTypeTraitsTest, is_const, UnqualObjectTypes) {

@lntue lntue merged commit 17b5d6b into llvm:main Mar 22, 2025
14 of 17 checks passed
mizvekov added a commit that referenced this pull request Mar 22, 2025
This moves the diagnostic for member pointers pointing into
non-class into BuildMemberPointer, so that it can be
used from RebuildMemberPointer, when instantiating templates.

Also adds a minor tweak to the diagnostic when the member pointer is anonymous,
which was previously untested.

Also reverts #132501, which
disabled a failing test due to the regression which is now fixed.

No changelog, since this fixes a regression which has not been released
yet.

Fixes #132494
@lntue lntue deleted the typetrait branch July 18, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants