================
@@ -4478,14 +4478,20 @@ renderDebugOptions(const ToolChain &TC, const Driver 
&D, const llvm::Triple &T,
       Args.getLastArg(options::OPT_ggnu_pubnames, 
options::OPT_gno_gnu_pubnames,
                       options::OPT_gpubnames, options::OPT_gno_pubnames);
   if (DwarfFission != DwarfFissionKind::None ||
-      (PubnamesArg && checkDebugInfoOption(PubnamesArg, Args, D, TC)))
-    if (!PubnamesArg ||
-        (!PubnamesArg->getOption().matches(options::OPT_gno_gnu_pubnames) &&
-         !PubnamesArg->getOption().matches(options::OPT_gno_pubnames)))
+      (PubnamesArg && checkDebugInfoOption(PubnamesArg, Args, D, TC))) {
+    const bool OptionSet =
----------------
MaskRay wrote:

The title reads as `-ggnu-pubnames` is removed for all DWARF v5 split-dwarf 
configurations, but the code seems to say lldb tuning. The title is worth 
clarification.

https://github.com/llvm/llvm-project/pull/83331
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to