Skip to content

Conversation

vvereschaka
Copy link
Contributor

Replaced non-existent %clang++ substitute with %clangxx. %clang++ gets became clang.exe++ on the Windows host that failures the test execution.

Replaced non-existent %clang++ substitute with %clangxx.
@llvmbot
Copy link
Member

llvmbot commented Aug 26, 2025

@llvm/pr-subscribers-bolt

Author: Vladimir Vereschaka (vvereschaka)

Changes

Replaced non-existent %clang++ substitute with %clangxx. %clang++ gets became clang.exe++ on the Windows host that failures the test execution.


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

1 Files Affected:

  • (modified) bolt/test/dump-dot-func.test (+1-1)
diff --git a/bolt/test/dump-dot-func.test b/bolt/test/dump-dot-func.test
index 510713dde6167..f05bfc195db45 100644
--- a/bolt/test/dump-dot-func.test
+++ b/bolt/test/dump-dot-func.test
@@ -1,7 +1,7 @@
 # Test the --dump-dot-func option with multiple functions 
 # (includes tests for both mangled/unmangled names)
 
-RUN: %clang++ %p/Inputs/multi-func.cpp -o %t.exe -Wl,-q
+RUN: %clangxx %p/Inputs/multi-func.cpp -o %t.exe -Wl,-q
 
 # Test 1: --dump-dot-func with specific function name (mangled)
 RUN: llvm-bolt %t.exe -o %t.bolt1 --dump-dot-func=_Z3addii -v=1 2>&1 | FileCheck %s --check-prefix=ADD

Copy link
Contributor

@maksfb maksfb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@maksfb maksfb changed the title [BOLT][test] Fix 'dump-dot-func' test execution on Windows host. [BOLT][test] Fix 'dump-dot-func' test execution on Windows host Aug 27, 2025
@vvereschaka vvereschaka merged commit 27e880f into llvm:main Aug 27, 2025
11 checks passed
@vvereschaka vvereschaka deleted the fix-bold-dump-dot-func-test branch August 27, 2025 21:04
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