Skip to content

[NFC][Clang] Don't check hardcode op num #135375

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jsji
Copy link
Member

@jsji jsji commented Apr 11, 2025

The num will change for any downstream customization.

The num will change for any downstream customization.
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:modules C++20 modules and Clang Header Modules labels Apr 11, 2025
@jsji jsji self-assigned this Apr 11, 2025
@jsji jsji requested review from cmc-rep and Maetveis April 11, 2025 14:22
@llvmbot
Copy link
Member

llvmbot commented Apr 11, 2025

@llvm/pr-subscribers-clang-modules

Author: Jinsong Ji (jsji)

Changes

The num will change for any downstream customization.


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

1 Files Affected:

  • (modified) clang/test/Modules/no-external-type-id.cppm (+1-1)
diff --git a/clang/test/Modules/no-external-type-id.cppm b/clang/test/Modules/no-external-type-id.cppm
index d067e574e72e3..2c05769e4f082 100644
--- a/clang/test/Modules/no-external-type-id.cppm
+++ b/clang/test/Modules/no-external-type-id.cppm
@@ -23,7 +23,7 @@ export module b;
 import a;
 export int b();
 
-// CHECK: <DECL_FUNCTION {{.*}} op8=4120
+// CHECK: <DECL_FUNCTION {{.*}} op8=[[#]]
 // CHECK: <TYPE_FUNCTION_PROTO
 
 //--- a.v1.cppm

@llvmbot
Copy link
Member

llvmbot commented Apr 11, 2025

@llvm/pr-subscribers-clang

Author: Jinsong Ji (jsji)

Changes

The num will change for any downstream customization.


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

1 Files Affected:

  • (modified) clang/test/Modules/no-external-type-id.cppm (+1-1)
diff --git a/clang/test/Modules/no-external-type-id.cppm b/clang/test/Modules/no-external-type-id.cppm
index d067e574e72e3..2c05769e4f082 100644
--- a/clang/test/Modules/no-external-type-id.cppm
+++ b/clang/test/Modules/no-external-type-id.cppm
@@ -23,7 +23,7 @@ export module b;
 import a;
 export int b();
 
-// CHECK: <DECL_FUNCTION {{.*}} op8=4120
+// CHECK: <DECL_FUNCTION {{.*}} op8=[[#]]
 // CHECK: <TYPE_FUNCTION_PROTO
 
 //--- a.v1.cppm

@Maetveis Maetveis requested a review from ChuanqiXu9 April 12, 2025 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:modules C++20 modules and Clang Header Modules clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants