[llvm] [lldb] [clang] [clang] Split out DebugOptions.def into its own top-level options group. (PR #75530)

Jan Svoboda via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 15 12:18:26 PST 2023


================
@@ -224,19 +233,20 @@ class CompilerInvocation : public CompilerInvocationBase {
   /// @{
   // Note: These need to be pulled in manually. Otherwise, they get hidden by
   // the mutable getters with the same names.
-  using CompilerInvocationBase::getLangOpts;
-  using CompilerInvocationBase::getTargetOpts;
-  using CompilerInvocationBase::getDiagnosticOpts;
-  using CompilerInvocationBase::getHeaderSearchOpts;
-  using CompilerInvocationBase::getPreprocessorOpts;
   using CompilerInvocationBase::getAnalyzerOpts;
-  using CompilerInvocationBase::getMigratorOpts;
   using CompilerInvocationBase::getAPINotesOpts;
   using CompilerInvocationBase::getCodeGenOpts;
+  using CompilerInvocationBase::getDebugOpts;
----------------
jansvoboda11 wrote:

Can you undo the reordering here? Let's just add the debug options into the right place and then optionally extract this change into a separate trivially NFC commit (and maybe reorder the member variables to match).

https://github.com/llvm/llvm-project/pull/75530


More information about the cfe-commits mailing list