ruiu added inline comments.
================ Comment at: lld/ELF/Options.td:361 +def time_trace: F<"time-trace">, HelpText<"Record time trace">; + ---------------- aganea wrote: > Given this option is a candidate for the other LLD drivers, I am wondering if > we couldn't have a shared `lld/Common/CommonOpt.td`. @ruiu WDYT? In many cases command line options are slightly different among drivers. so currently we have a completely separated option file for each driver. It looks like maintaining each file separately makes things much easier, so I wouldn't create a new common file. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69043/new/ https://reviews.llvm.org/D69043 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
