Skip to content

Commit 6245406

Browse files
DenisBakhvalovDenisBakhvalov
committed
[ESIMD] Remove -fsycl-explicit-simd option
ESIMD no longer requires -fsycl-explicit-simd flag, so it can be removed.
1 parent 26f1c0a commit 6245406

26 files changed

+25
-52
lines changed

‎clang/include/clang/Driver/Options.td

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4370,10 +4370,6 @@ def fsycl : Flag<["-"], "fsycl">, Flags<[NoXarchOption, CoreOption]>, Group<sycl
43704370
HelpText<"Enables SYCL kernels compilation for device">;
43714371
def fno_sycl : Flag<["-"], "fno-sycl">, Flags<[NoXarchOption, CoreOption]>, Group<sycl_Group>,
43724372
HelpText<"Disables SYCL kernels compilation for device">;
4373-
defm sycl_esimd: BoolFOption<"sycl-explicit-simd",
4374-
LangOpts<"SYCLExplicitSIMD">, DefaultFalse,
4375-
PosFlag<SetTrue, [CC1Option], "Enable">, NegFlag<SetFalse, [], "Disable">,
4376-
BothFlags<[NoArgumentUnused, CoreOption], "SYCL explicit SIMD extension.">>;
43774373
defm sycl_early_optimizations : OptOutFFlag<"sycl-early-optimizations", "Enable", "Disable", " standard optimization pipeline for SYCL device compiler", [CoreOption]>;
43784374
def fsycl_dead_args_optimization : Flag<["-"], "fsycl-dead-args-optimization">,
43794375
Group<sycl_Group>, Flags<[NoArgumentUnused, CoreOption]>, HelpText<"Enables "

‎clang/lib/Driver/ToolChains/Clang.cpp

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4344,10 +4344,6 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
43444344
CmdArgs.push_back("-fsycl-is-device");
43454345
CmdArgs.push_back("-fdeclare-spirv-builtins");
43464346

4347-
if (Args.hasFlag(options::OPT_fsycl_esimd, options::OPT_fno_sycl_esimd,
4348-
false))
4349-
CmdArgs.push_back("-fsycl-explicit-simd");
4350-
43514347
// Default value for FPGA is false, for all other targets is true.
43524348
if (!Args.hasFlag(options::OPT_fsycl_early_optimizations,
43534349
options::OPT_fno_sycl_early_optimizations,
@@ -6638,10 +6634,6 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
66386634
// doing the host pass.
66396635
CmdArgs.push_back("-fsycl-is-host");
66406636

6641-
if (Args.hasFlag(options::OPT_fsycl_esimd, options::OPT_fno_sycl_esimd,
6642-
false))
6643-
CmdArgs.push_back("-fsycl-explicit-simd");
6644-
66456637
if (!D.IsCLMode()) {
66466638
// SYCL library is guaranteed to work correctly only with dynamic
66476639
// MSVC runtime.

‎clang/test/Driver/sycl-esimd.cpp

Lines changed: 0 additions & 10 deletions
This file was deleted.

‎clang/test/Driver/sycl-offload.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -596,7 +596,6 @@
596596

597597
/// Check "-spirv-allow-unknown-intrinsics=llvm.genx." option is emitted for llvm-spirv tool
598598
// RUN: %clangxx %s -fsycl -### 2>&1 | FileCheck %s --check-prefix=CHK-ALLOW-INTRIN
599-
// RUN: %clangxx %s -fsycl -fsycl-explicit-simd -### 2>&1 | FileCheck %s --check-prefix=CHK-ALLOW-INTRIN
600599
// CHK-ALLOW-INTRIN: llvm-spirv{{.*}}-spirv-allow-unknown-intrinsics=llvm.genx.
601600

602601
/// ###########################################################################

‎clang/test/SemaSYCL/esimd-globals-in-sycl-context.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clang_cc1 -fsycl-is-device -fsycl-explicit-simd -fsyntax-only -verify %s
1+
// RUN: %clang_cc1 -fsycl-is-device -fsyntax-only -verify %s
22

33
// This test checks usage of an ESIMD global in ESIMD(positive) and SYCL(negative) contexts.
44

‎sycl/test/esimd/block_load_store.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl.hpp>

‎sycl/test/esimd/esimd-util-compiler-eval.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -c %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -c %s
22
// This test checks compile-time evaluation of functions from esimd_util.hpp
33

44
#include "CL/sycl.hpp"

‎sycl/test/esimd/esimd_math.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl/INTEL/esimd.hpp>

‎sycl/test/esimd/flat_atomic.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl.hpp>

‎sycl/test/esimd/gather4_scatter4.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl.hpp>

‎sycl/test/esimd/gather_scatter.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl.hpp>

‎sycl/test/esimd/glob.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -c -fsycl-device-only -Xclang -emit-llvm %s -o %t
1+
// RUN: %clangxx -fsycl -c -fsycl-device-only -Xclang -emit-llvm %s -o %t
22
// RUN: sycl-post-link -split-esimd -lower-esimd -O2 -S %t -o %t.table
33
// RUN: FileCheck %s -input-file=%t_esimd_0.ll
44

‎sycl/test/esimd/global_var.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22

33
#include <CL/sycl.hpp>
44
#include <CL/sycl/INTEL/esimd.hpp>

‎sycl/test/esimd/hw_compile.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
// Basic ESIMD test which checks that ESIMD invocation syntax can get compiled.
2-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -c %s -o %t.bc
2+
// RUN: %clangxx -fsycl -fsycl-device-only -c %s -o %t.bc
33

44
#include <CL/sycl.hpp>
55
#include <CL/sycl/INTEL/esimd.hpp>

‎sycl/test/esimd/intrins_trans.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -O0 -fsycl -fsycl-explicit-simd -fsycl-device-only -Xclang -emit-llvm %s -o %t
1+
// RUN: %clangxx -O0 -fsycl -fsycl-device-only -Xclang -emit-llvm %s -o %t
22
// RUN: sycl-post-link -split-esimd -lower-esimd -O0 -S %t -o %t.table
33
// RUN: FileCheck %s -input-file=%t_esimd_0.ll
44

‎sycl/test/esimd/odr.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@
44
// Template functions must have the same instantiation in both sources to cause
55
// ODR problems potentially - esimd_min is used for that purpose.
66
//
7-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-targets=%sycl_triple -DSOURCE1 -c %s -o %t1.o
8-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-targets=%sycl_triple -DSOURCE2 -c %s -o %t2.o
9-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-targets=%sycl_triple %t1.o %t2.o -o %t.exe
7+
// RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple -DSOURCE1 -c %s -o %t1.o
8+
// RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple -DSOURCE2 -c %s -o %t2.o
9+
// RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %t1.o %t2.o -o %t.exe
1010
//
1111
// Cuda does not support intrinsics generated by the ESIMD compilation path:
1212
// UNSUPPORTED: cuda

‎sycl/test/esimd/simd.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl/INTEL/esimd.hpp>

‎sycl/test/esimd/simd_merge.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl/INTEL/esimd.hpp>

‎sycl/test/esimd/simd_view.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl/INTEL/esimd.hpp>

‎sycl/test/esimd/slm_atomic.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl.hpp>

‎sycl/test/esimd/slm_block.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl.hpp>

‎sycl/test/esimd/slm_load.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl.hpp>

‎sycl/test/esimd/slm_load4.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -fsyntax-only -Xclang -verify %s
1+
// RUN: %clangxx -fsycl -fsycl-device-only -fsyntax-only -Xclang -verify %s
22
// expected-no-diagnostics
33

44
#include <CL/sycl.hpp>

‎sycl/test/esimd/spirv_intrins_trans.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd -fsycl-device-only -S -emit-llvm -x c++ %s -o %t
1+
// RUN: %clangxx -fsycl -fsycl-device-only -S -emit-llvm -x c++ %s -o %t
22
// RUN: sycl-post-link -split-esimd -lower-esimd -O0 -S %t -o %t.table
33
// RUN: FileCheck %s -input-file=%t_esimd_0.ll
44

‎sycl/test/esimd/vadd.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
1-
// RUN: %clangxx -fsycl -fsycl-explicit-simd %s -o %t.out
1+
// RUN: %clangxx -fsycl %s -o %t.out
22
// RUN: %RUN_ON_HOST %t.out
33

44
// Check that the code compiles with -O0 and -g
5-
// RUN: %clangxx -I %sycl_include %s -o %t.out -fsycl -fsycl-explicit-simd -O0
6-
// RUN: %clangxx -I %sycl_include %s -o %t.out -fsycl -fsycl-explicit-simd -O0 -g
5+
// RUN: %clangxx -I %sycl_include %s -o %t.out -fsycl -O0
6+
// RUN: %clangxx -I %sycl_include %s -o %t.out -fsycl -O0 -g
77

88
#include <CL/sycl.hpp>
99
#include <CL/sycl/INTEL/esimd.hpp>

‎sycl/test/on-device/lit.cfg.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -225,10 +225,6 @@ def getDeviceCount(device_type):
225225
if platform.system() == "Linux":
226226
gpu_run_on_linux_substitute = "env SYCL_DEVICE_FILTER={SYCL_PLUGIN}:gpu ".format(SYCL_PLUGIN=backend)
227227
gpu_check_on_linux_substitute = "| FileCheck %s"
228-
# ESIMD-specific setup. Requires OpenCL for now.
229-
esimd_run_substitute = " env SYCL_DEVICE_FILTER=opencl:gpu SYCL_PROGRAM_COMPILE_OPTIONS=-vc-codegen"
230-
config.substitutions.append( ('%ESIMD_RUN_PLACEHOLDER', esimd_run_substitute) )
231-
config.substitutions.append( ('%clangxx-esimd', "clang++ -fsycl-explicit-simd" ) )
232228
else:
233229
lit_config.warning("GPU device not found")
234230

0 commit comments

Comments
 (0)