tejohnson added inline comments.
================ Comment at: llvm/include/llvm/LTO/Config.h:92 + /// The name for the split debug info file used for the DW_AT_[GNU_]dwo_name + /// attribute in the skeleton CU. + std::string SplitDwarfFile; ---------------- aaronpuchert wrote: > tejohnson wrote: > > Probably needs a comment similar to the one below about being for running > > individual backends directly. Otherwise (for in process ThinLTO) we use the > > DwoDir. > Sure. Any objections to the following wording? > > ``` > /// The name for the split debug info file used for the DW_AT_[GNU_]dwo_name > /// attribute in the skeleton CU. This should generally only be used when > /// running an individual backend directly via thinBackend(), as otherwise > /// all objects would use the same .dwo file. Not used as output path. > ``` Looks good. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59673/new/ https://reviews.llvm.org/D59673 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits