saiislam marked an inline comment as done. saiislam added inline comments.
================ Comment at: clang/docs/ClangOffloadBundler.rst:296 + * Their target triple are the same + * Their GPUArch are the same ---------------- yaxunl wrote: > This does not match our final view of compatibility. Is this intentional? > Thanks. Yes, I have intentionally removed the TargetID related compatibility testing. Plan is to introduce it with an upcoming patch for supporting TargetID in OpenMP. So, all changes in the clang driver and offload-bundler for TargetID will be introduced in one go. D106870 is supposed to put most of the necessary infrastructure in place so that TargetID support can be cleanly added. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110083/new/ https://reviews.llvm.org/D110083 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
