Skip to content

Commit 0b72b71

Browse files
committed
[Dexter] Fix incorrect substitution errors in clang-cl builder
Following 262520a, tests on windows bots began failing due to an incorrect substitution in the previous patch, where clang-cl was used instead of clang_cl. Also fixes an inconsistency in the builders used for some of the tests in 'dexter-tests', where %clang++ was used for some tests and %clang for tests that should have identical RUN lines.
1 parent f0bbda0 commit 0b72b71

File tree

7 files changed

+9
-9
lines changed

7 files changed

+9
-9
lines changed

cross-project-tests/debuginfo-tests/dexter-tests/global-constant.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// REQUIRES: system-windows
22
//
3-
// RUN: %clang-cl /Z7 /Zi %s -o %t
3+
// RUN: %clang_cl /Z7 /Zi %s -o %t
44
// RUN: %dexter --fail-lt 1.0 -w --binary %t --debugger 'dbgeng' -- %s
55

66
// Check that global constants have debug info.

cross-project-tests/debuginfo-tests/dexter-tests/hello.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// REQUIRES: system-windows
22
//
3-
// RUN: %clang-cl /Z7 /Zi %s -o %t
3+
// RUN: %clang_cl /Z7 /Zi %s -o %t
44
// RUN: %dexter --fail-lt 1.0 -w --binary %t --debugger 'dbgeng' -- %s
55

66
#include <stdio.h>

cross-project-tests/debuginfo-tests/dexter-tests/inline-line-gap.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
// REQUIRES: system-windows
22
//
3-
// RUN: %clang-cl /Od /Z7 /Zi %s -o %t
3+
// RUN: %clang_cl /Od /Z7 /Zi %s -o %t
44
// RUN: %dexter --fail-lt 1.0 -w --binary %t --debugger 'dbgeng' -- %s
55
//
6-
// RUN: %clang-cl /O2 /Z7 /Zi %s -o %t
6+
// RUN: %clang_cl /O2 /Z7 /Zi %s -o %t
77
// RUN: %dexter --fail-lt 1.0 -w --binary %t \
88
// RUN: --debugger 'dbgeng' -- %s
99

cross-project-tests/debuginfo-tests/dexter-tests/nrvo.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
//
44
// REQUIRES: system-windows
55
//
6-
// RUN: %clang-cl /Z7 /Zi %s -o %t
6+
// RUN: %clang_cl /Z7 /Zi %s -o %t
77
// RUN: %dexter --fail-lt 1.0 -w --binary %t --debugger 'dbgeng' -- %s
88

99
struct string {

cross-project-tests/debuginfo-tests/dexter-tests/optnone-vectors-and-functions.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
// RUN: %clang -O2 -g %s -o %t
1+
// RUN: %clang++ -O2 -g %s -o %t
22
// RUN: %dexter --fail-lt 1.0 -w \
33
// RUN: --binary %t --debugger 'lldb' -v -- %s
4-
// RUN: %clang -O0 -g %s -o %t
4+
// RUN: %clang++ -O0 -g %s -o %t
55
// RUN: %dexter --fail-lt 1.0 -w \
66
// RUN: --binary %t --debugger 'lldb' -- %s
77

cross-project-tests/debuginfo-tests/dexter-tests/realigned-frame.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// REQUIRES: system-windows
22
//
3-
// RUN: %clang-cl /Z7 /Zi %s -o %t
3+
// RUN: %clang_cl /Z7 /Zi %s -o %t
44
// RUN: %dexter --fail-lt 1.0 -w --binary %t --debugger 'dbgeng' -- %s
55

66
// From https://llvm.org/pr38857, where we had issues with stack realignment.

cross-project-tests/lit.cfg.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ def configure_dexter_substitutions():
123123
if platform.system() == "Windows":
124124
# The Windows builder script uses lld.
125125
dependencies = ["clang", "lld-link"]
126-
dexter_regression_test_builder = "clang-cl"
126+
dexter_regression_test_builder = "clang_cl"
127127
dexter_regression_test_debugger = "dbgeng"
128128
dexter_regression_test_flags = "/Zi /Od"
129129
else:

0 commit comments

Comments
 (0)