[PATCH] D99190: WIP: [SYCL] Add design document for SYCL mode

[Alexey Bataev via Phabricator via cfe-commits]

ABataev added inline comments.


================
Comment at: clang/docs/SYCLSupport.md:123
+traverse all symbols accessible from kernel functions and add them to the
+"device part" of the code marking them with the new SYCL device attribute.
+
----------------
bader wrote:
> Naghasan wrote:
> > OpenMP offload uses a similar approach isn't it? Might be worth to describe how the 2 relates to each other and where they diverge. 
> Do you mean the approach OpenMP compiler uses to outline single-source code parts to offload?
> To be honest, I'm not sure... @ABataev, is there any description how OpenMP compiler outlines device code?
> https://clang.llvm.org/docs/OpenMPSupport.html doesn't provide much details unfortunately.
I don't think we have anything like this. Moreover, currently, there are 2 different models, one with outlining by the frontend and another one with the outlining by the LLVM.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99190/new/

https://reviews.llvm.org/D99190