[PATCH] D110083: [clang-offload-bundler][docs][NFC] Add archive unbundling documentation

Mon Sep 20 12:20:22 PDT 2021

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