[Driver][SYCL] Add support for -fsyclbin

[mdtoguchi]

Adds support for the -fsyclbin option. This option allows for creation of a SYCL device specific binary. The binary is created via a device compilation before being packaged and sent to the clang-linker-wrapper. The clang-linker-wrapper is responsible to take the device 'object' and create the corresponding .syclbin file.

[mdtoguchi]

Not ready for prime-time yet. The corresponding support in the clang-linker-wrapper is not in place.

[steffenlarsen]

Overall I think this is great and I have tested it locally with the changes from #16873.

I do wonder if we could let -fsyclbin imply -fsycl, as I don't see there being a use-case where you wouldn't have both, but I really don't think it's a big issue either way.

[mdtoguchi]

Overall I think this is great and I have tested it locally with the changes from #16873.

I do wonder if we could let -fsyclbin imply -fsycl, as I don't see there being a use-case where you wouldn't have both, but I really don't think it's a big issue either way.

Adding the implication is reasonable - similar to -fsycl-device-only. Maybe update the design to reflect?

[steffenlarsen]

Overall I think this is great and I have tested it locally with the changes from #16873.
I do wonder if we could let -fsyclbin imply -fsycl, as I don't see there being a use-case where you wouldn't have both, but I really don't think it's a big issue either way.

Adding the implication is reasonable - similar to -fsycl-device-only. Maybe update the design to reflect?

Done!

[steffenlarsen]

@mdtoguchi - #16873 has been merged. You can, if you want, change the 3 first RUN lines in sycl/test/syclbin/simple_kernel.cpp to:

// RUN: %clangxx -fsyclbin -o %t.syclbin %t.out

[maarquitos14]

LGTM, just a couple of nits.

[mdtoguchi]

@intel/llvm-gatekeepers, this is ready for merge. Thanks!