asudarsa wrote: Hi @llvm-beanz
Thanks so much for providing your valuable time and feedback thus far. I have addressed the concerns raised to the best of my ability. I have added more testing inside 'clang/test/Driver/clang-sycl-linker-test.cpp'. It will be great if you can take another look when convenient and let me know if there are additional concerns. Thanks Sincerely P.S: About the location of this tool under clang/tools, I added the tool here as it is somewhat similar in functionality to tools like clang-linker-wrapper and clang-nvlink-wrapper. If there is strong opposition, we can surely move it to llvm/tools. https://github.com/llvm/llvm-project/pull/112245 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
