Skip to content

Commit d826c04

Browse files
committed
[SYCL] Make SPV_INTEL_runtime_aligned be added by driver only for FPGA H/W
Signed-off-by: Dmitry Sidorov <dmitry.sidorov@intel.com>
1 parent 1e55cf3 commit d826c04

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
@@ -9041,17 +9041,12 @@ void SPIRVTranslator::ConstructJob(Compilation &C, const JobAction &JA,
90419041
",+SPV_INTEL_long_constant_composite"
90429042
",+SPV_INTEL_fpga_invocation_pipelining_attributes"
90439043
",+SPV_INTEL_fpga_dsp_control"
9044-
",+SPV_INTEL_arithmetic_fence"
9045-
",+SPV_INTEL_runtime_aligned";
9044+
",+SPV_INTEL_arithmetic_fence";
90469045
ExtArg = ExtArg + DefaultExtArg + INTELExtArg;
90479046
if (!C.getDriver().isFPGAEmulationMode())
9048-
// Enable SPV_INTEL_usm_storage_classes only for FPGA hardware,
9049-
// since it adds new storage classes that represent global_device and
9050-
// global_host address spaces, which are not supported for all
9051-
// targets. With the extension disabled the storage classes will be
9052-
// lowered to CrossWorkgroup storage class that is mapped to just
9053-
// global address space.
9054-
ExtArg += ",+SPV_INTEL_usm_storage_classes";
9047+
// Enable several extensions on FPGA H/W exclusively
9048+
ExtArg += ",+SPV_INTEL_usm_storage_classes"
9049+
",+SPV_INTEL_runtime_aligned";
90559050
else
90569051
// Don't enable several freshly added extensions on FPGA H/W
90579052
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)