maksimsab wrote:

@jhuber6
I had a choice to put it the common `OffloadWrapper.{h,cpp}` area. But SYCL's 
offload wrapping looks very different from others and I decided not to mix them 
together. Hence, I was choosing between 
`llvm/Frontend/Offloading/SYCLOffloadWrapper.h` and 
`llvm/Frontend/SYCL/OffloadWrapper.h`. If putting it together in 
`llvm/Frontend/Offloading/OffloadWrapper.h` looks better to you then I don't 
object to that.

https://github.com/llvm/llvm-project/pull/147508
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to