yaxunl added inline comments.
================ Comment at: clang/lib/Driver/Driver.cpp:4099-4102 + for (auto &Arg : Args.getAllArgValues(options::OPT_offload_arch_EQ)) + Archs.insert(getCanonicalArchString(C, Args, Arg, Kind)); + for (auto &Arg : Args.getAllArgValues(options::OPT_no_offload_arch_EQ)) + Archs.erase(getCanonicalArchString(C, Args, Arg, Kind)); ---------------- The final set depends on the order of -offload-arch and -no-offload-arch options, e.g. `--offload-arch=gfx906 --no-offload-arch=gfx906` and `--no-offload-arch=gfx906 --offload-arch=gfx906` is different. Also there is `--no-offload-arch=all` which removes all precedent --offload-arch= options. ================ Comment at: clang/lib/Driver/Driver.cpp:4107 + Archs.insert(CudaArchToString(CudaArch::SM_35)); + else if (Kind == Action::OFK_Cuda) + Archs.insert(CudaArchToString(CudaArch::GFX803)); ---------------- should this be HIP? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120272/new/ https://reviews.llvm.org/D120272 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
