2021 04 12 sync

Meeting notes

• Status update
  • Alexey: The SYCL documentation section with address space attributes behavior implementation has been approved - https://reviews.llvm.org/D99488. It depends on https://reviews.llvm.org/D99190 review request creating a landing page for SYCL documentation and filling SYCL device compiler section. https://reviews.llvm.org/D99190 has been approved by Victor, but there was a request for changes from Ronan, so we need additional approve.
  • AR for Alexey: re-order patches with the documentation commit already approved https://reviews.llvm.org/D99488 first.
  • Last week we discussed that we should consider using CUDA/OpenMP compiler design with customized host compiler instead of the integration header developed for SYCL. As https://reviews.llvm.org/D99190 describes an integration header design, which is being under discussion for upstream, we agreed to make it explicit in the documentation that integration header design is
  • We discussed that customizing SYCL host compiler might be useful for some user scenarios (e.g. handling SYCL-specific C++ attributes). On the other hand, there are users who are interested in using DC++ with vanilla C++ compiler (e.g. gcc).
  • Anastasia left a number of comments for the implementation and base on her feedback, we created https://github.com/intel/llvm/issues/3534 to investigate if we can avoid adding -sycldevice target triple environment component.
  • https://github.com/intel/llvm/pull/3535 should remove uses of the -sycldevice component. Victor noted that SYCL-specific passes added for NVPTX target rely on corresponding runtime environment adjustments. Current patch should be okay, but in addition to that we can consider adding named module-level metadata. The same applies to dead kernel parameter elimination pass.
  • libclc has two hack: one in cmake files to work-around the issue #1814 and another one for the difference in name mangling. We should be able to link LLVM modules compiled by OpenCL and SYCL front-ends. Codeplay team is working on removing these hacks.
  • Rest of the -sycldevice component uses can be removed from DPC++ implementation.