jdoerfert added a comment. Thanks for splitting this. I quickly went over it only.
================ Comment at: clang/docs/ClangOffloadBundler.rst:137 + + myarch--myOS- ---------------- not needed here. ================ Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:1431 + ? UnbundleArchive() + : Unbundle ? UnbundleFiles() : BundleFiles(); + }); ---------------- By now, a proper conditional seems appropriate. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93525/new/ https://reviews.llvm.org/D93525 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
