[clang] [llvm] [SYCL] Add clang-linker-wrapper changes to call clang-sycl-linker for SYCL offloads (PR #135683)
Arvind Sudarsanam via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 14 16:18:28 PDT 2025
================
@@ -464,7 +464,8 @@ fatbinary(ArrayRef<std::pair<StringRef, StringRef>> InputFiles,
} // namespace amdgcn
namespace generic {
-Expected<StringRef> clang(ArrayRef<StringRef> InputFiles, const ArgList &Args) {
+Expected<StringRef> clang(ArrayRef<StringRef> InputFiles, const ArgList &Args,
+ bool HasSYCLOffloadKind = false) {
----------------
asudarsa wrote:
I agree this change makes things better. Making OffloadKind enums to have power-of-two values and then using a bitfield here helps to avoid these flags and also helps to make the ActiveOffloadKinds support better. However, I would prefer to make these changes in a subsequent PR so that we can have a set of simpler commits. Please let me know if this is agreeable or you would prefer to make the change in this PR.
Thanks
https://github.com/llvm/llvm-project/pull/135683
More information about the cfe-commits
mailing list