compnerd added inline comments.
================ Comment at: lld/ELF/InputFiles.cpp:662 } + case SHT_LLVM_DEPLIBS: { + if (Config->Relocatable) ---------------- Can you make the flag here reflect the name as well? (`SHT_LLVM_DEPENDENT_LIBRARIES`) ================ Comment at: lld/ELF/Options.td:74 +defm deplibs: B<"deplibs", + "Process dependent library specifiers from input files (default)", ---------------- I think that I prefer the `--dependent-libraries` flag which is similar to the recent additions to the linker options. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60274/new/ https://reviews.llvm.org/D60274 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits