hvdijk wrote: > I don't know what others would think about this, but without the subsequent > changes that make use of this, I don't think this should be landed in LLVM. > After all, plans might change, the community might refuse the subsequent PRs > etc, resulting in dead code lying around.
I'm not going to argue strongly -- after all, in the upstream PR, I did say I considered submitting this, decided against it, and was asked to submit this anyway -- but if plans change, if DPC++ either never submits its libsycl or LLVM rejects it, LLVM will still have to come up with some other SYCL runtime and that will still need this PR. https://github.com/llvm/llvm-project/pull/118635 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
