rsmith added inline comments.
================ Comment at: clang/test/Driver/clang-offload-bundler.c:390 +// RUN: clang-offload-bundler -unbundle -type=a -targets=openmp-amdgcn-amd-amdhsa--gfx906,openmp-amdgcn-amd-amdhsa--gfx908 -inputs=%t.input-archive.a -outputs=%t-archive-gfx906-simple.a,%t-archive-gfx908-simple.a +// RUN: llvm-ar t %t-archive-gfx906-simple.a | FileCheck %s -check-prefix=GFX906 +// GFX906: simple-openmp-amdgcn-amd-amdhsa--gfx906 ---------------- This test does not depend on `llvm-ar`, and this change causes `check-clang` to fail in the case where `llvm-ar` has not previously been built. Please can you fix? (Might need some changes to the build rules to add a dependency on `llvm-ar`, if you can't avoid depending on it for this test.) 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