Skip to content

Commit 5f3b004

Browse files
authored
Merge pull request intel#1198 from myler/CMPLRTST-17973
CMPLRTST-17973: switch icx/icpx instead of dpcpp to run llvm_test_suite_sycl
2 parents 7ebe66c + 4a602ea commit 5f3b004

File tree

1 file changed

+19
-13
lines changed

1 file changed

+19
-13
lines changed

llvm_test_suite_sycl.pl

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ sub init_test
162162
if (defined $ENV{BASECOMPILER}) {
163163
$compiler_path = $ENV{BASECOMPILER};
164164
} else {
165-
$compiler_path = generate_tool_path("dpcpp", "clang++");
165+
$compiler_path = generate_tool_path("icx", "clang++");
166166
}
167167

168168
my $branch = "";
@@ -697,7 +697,13 @@ sub set_tool_path
697697

698698
# For the product compiler, add the internal "bin-llvm" directory to PATH.
699699
if ($compiler =~ /xmain/) {
700-
my $llvm_dir = dirname(qx/dpcpp -print-prog-name=llvm-ar/);
700+
my $llvm_dir = "";
701+
if ($cmplr_platform{OSFamily} eq "Windows") {
702+
$llvm_dir = dirname(qx/which llvm-ar/);
703+
} else {
704+
$llvm_dir = dirname(qx/icx -print-prog-name=llvm-ar/);
705+
}
706+
chomp $llvm_dir;
701707
my $llvm_path = join($path_sep, $llvm_dir, $ENV{PATH});
702708
set_envvar("PATH", $llvm_path, join($path_sep, $llvm_dir, '$PATH'));
703709
}
@@ -819,13 +825,13 @@ sub generate_tool_path
819825
my $driver = shift;
820826
my $tool_name = shift;
821827

822-
my $tool_path = qx/$driver --print-prog-name=$tool_name/;
823-
chomp $tool_path;
824-
828+
my $tool_path;
825829
if ($cmplr_platform{OSFamily} eq "Windows") {
826-
$tool_path =~ tr#\\#/#;
827-
$tool_path = "$tool_path.exe";
830+
$tool_path = qx/which $tool_name/;
831+
} else {
832+
$tool_path = qx/$driver --print-prog-name=$tool_name/;
828833
}
834+
chomp $tool_path;
829835

830836
return $tool_path;
831837
}
@@ -857,10 +863,10 @@ sub run_cmake
857863
} else {
858864
$c_cmplr = "clang";
859865
$cpp_cmplr = 'clang++';
860-
# Clang is not guaranteed to be in PATH, but dpcpp is. Ask dpcpp
866+
# Clang is not guaranteed to be in PATH, but icx is. Ask icx
861867
# for absolute paths.
862-
$c_cmplr = generate_tool_path("dpcpp", $c_cmplr);
863-
$cpp_cmplr = generate_tool_path("dpcpp", $cpp_cmplr);
868+
$c_cmplr = generate_tool_path("icx", $c_cmplr);
869+
$cpp_cmplr = generate_tool_path("icx", $cpp_cmplr);
864870
$c_cmd_opts = convert_opt($c_cmd_opts);
865871
$cpp_cmd_opts = convert_opt($cpp_cmd_opts);
866872
}
@@ -869,10 +875,10 @@ sub run_cmake
869875
$c_cmplr = "clang";
870876
if ($compiler =~ /xmain/) {
871877
$cpp_cmplr = "clang++";
872-
# Clang is not guaranteed to be in PATH, but dpcpp is. Ask dpcpp
878+
# Clang is not guaranteed to be in PATH, but icx is. Ask icx
873879
# for absolute paths.
874-
$c_cmplr = generate_tool_path("dpcpp", $c_cmplr);
875-
$cpp_cmplr = generate_tool_path("dpcpp", $cpp_cmplr);
880+
$c_cmplr = generate_tool_path("icx", $c_cmplr);
881+
$cpp_cmplr = generate_tool_path("icpx", $cpp_cmplr);
876882
}
877883
$thread_opts = "-lpthread";
878884
}

0 commit comments

Comments
 (0)