khchen added inline comments.
================ Comment at: clang/lib/Driver/ToolChains/Arch/RISCV.cpp:387 + +void riscv::addGoldOptions(const ToolChain &ToolChain, + const llvm::opt::ArgList &Args, ---------------- kito-cheng wrote: > MaskRay wrote: > > gold doesn't support RISC-V, does it? > Gold doesn't support RISC-V , but ld.bfd supported same plugin API, so this > made clang can run LTO with ld.bfd. I reference `AddGoldPlugin` to naming this function, is it confusing? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67409/new/ https://reviews.llvm.org/D67409 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits