-
Notifications
You must be signed in to change notification settings - Fork 130
[SYCL] [FPGA] Create DSP control E2E emulator test #591
Conversation
/verify with intel/llvm#5035 |
731134e
to
ca46da2
Compare
ca46da2
to
22d6bc3
Compare
/verify with intel/llvm#5035 |
Create header file that provides user API for FPGA DSP control feature. Related links: SPIR-V spec: https://github.com/KhronosGroup/SPIRV-Registry/blob/main/extensions/INTEL/SPV_INTEL_fpga_dsp_control.asciidoc SPIR-V implementation: [SPIRV] New FPGA function attribute for DSP control KhronosGroup/SPIRV-LLVM-Translator#1046 Test: intel/llvm-test-suite#591
Note that the new test Basic/fpga_tests/fpga_dsp_control.cpp passed in Jenkins/llvm-test-suite: http://llvm-ci-test2.intel.com:8080/blue/organizations/jenkins/SYCL_CI%2Fintel%2FLin%2FLLVM_Test_Suite_CUDA/detail/LLVM_Test_Suite_CUDA/80/pipeline/#step-239-log-156. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test is LGTM, just comments to the run strings
43bccd2
to
c9e4609
Compare
/verify with intel/llvm#5035 |
Both fpga_dsp_control.cpp and fpga_lsu.cpp passed compile and run in the latest Jenkins/llvm-test-suite. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shuoniu-intel Thank you.
Create E2E emulator test for newly added DSP control header.
The corresponding SYCL PR: intel/llvm#5035