================
@@ -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

Reply via email to