Skip to content

Commit 3651971

Browse files
authored
[SYCL] Enable SPV_INTEL_runtime_aligned for FPGA only (#5299)
Signed-off-by: Dmitry Sidorov <dmitry.sidorov@intel.com>
1 parent 459fa6f commit 3651971

File tree

2 files changed

+6
-12
lines changed

2 files changed

+6
-12
lines changed

clang/lib/Driver/ToolChains/Clang.cpp

+4-9
Original file line numberDiff line numberDiff line change
@@ -9044,17 +9044,12 @@ void SPIRVTranslator::ConstructJob(Compilation &C, const JobAction &JA,
90449044
",+SPV_INTEL_long_constant_composite"
90459045
",+SPV_INTEL_fpga_invocation_pipelining_attributes"
90469046
",+SPV_INTEL_fpga_dsp_control"
9047-
",+SPV_INTEL_arithmetic_fence"
9048-
",+SPV_INTEL_runtime_aligned";
9047+
",+SPV_INTEL_arithmetic_fence";
90499048
ExtArg = ExtArg + DefaultExtArg + INTELExtArg;
90509049
if (!C.getDriver().isFPGAEmulationMode())
9051-
// Enable SPV_INTEL_usm_storage_classes only for FPGA hardware,
9052-
// since it adds new storage classes that represent global_device and
9053-
// global_host address spaces, which are not supported for all
9054-
// targets. With the extension disabled the storage classes will be
9055-
// lowered to CrossWorkgroup storage class that is mapped to just
9056-
// global address space.
9057-
ExtArg += ",+SPV_INTEL_usm_storage_classes";
9050+
// Enable several extensions on FPGA H/W exclusively
9051+
ExtArg += ",+SPV_INTEL_usm_storage_classes"
9052+
",+SPV_INTEL_runtime_aligned";
90589053
else
90599054
// Don't enable several freshly added extensions on FPGA H/W
90609055
ExtArg += ",+SPV_INTEL_token_type"

clang/test/Driver/sycl-spirv-ext.c

+2-3
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@
5050
// CHECK-DEFAULT-SAME:,+SPV_INTEL_fpga_invocation_pipelining_attributes
5151
// CHECK-DEFAULT-SAME:,+SPV_INTEL_fpga_dsp_control
5252
// CHECK-DEFAULT-SAME:,+SPV_INTEL_arithmetic_fence
53-
// CHECK-DEFAULT-SAME:,+SPV_INTEL_runtime_aligned
5453
// CHECK-DEFAULT-SAME:,+SPV_INTEL_token_type
5554
// CHECK-DEFAULT-SAME:,+SPV_INTEL_bfloat16_conversion
5655
// CHECK-DEFAULT-SAME:,+SPV_INTEL_joint_matrix
@@ -80,5 +79,5 @@
8079
// CHECK-FPGA-HW-SAME:,+SPV_INTEL_fpga_invocation_pipelining_attributes
8180
// CHECK-FPGA-HW-SAME:,+SPV_INTEL_fpga_dsp_control
8281
// CHECK-FPGA-HW-SAME:,+SPV_INTEL_arithmetic_fence
83-
// CHECK-FPGA-HW-SAME:,+SPV_INTEL_runtime_aligned
84-
// CHECK-FPGA-HW-SAME:,+SPV_INTEL_usm_storage_classes"
82+
// CHECK-FPGA-HW-SAME:,+SPV_INTEL_usm_storage_classes
83+
// CHECK-FPGA-HW-SAME:,+SPV_INTEL_runtime_aligned"

0 commit comments

Comments
 (0)