================ @@ -481,9 +481,9 @@ ClangDocContext::ClangDocContext(tooling::ExecutionContext *ECtx, StringRef RepositoryUrl, StringRef RepositoryLinePrefix, StringRef Base, std::vector<std::string> UserStylesheets, - bool FTimeTrace) + bool KeepJSON, bool FTimeTrace) ---------------- ilovepi wrote:
does this actually need to be in the context? you have a `cl::opt`, right? it doesn't have to be a static variable, and then you could use it from other TUs. This is pretty common in LLVM. Alternatively, the `cl::opt` could just be in the mustache generator, since its the only generator that uses it. https://github.com/llvm/llvm-project/pull/150656 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits