dblaikie added inline comments.

================
Comment at: clang/include/clang/Driver/Options.td:1955
   Flags<[CC1Option]>;
+def fdwarf_default_version: Joined<["-"], "fdwarf-default-version=">, 
Group<f_Group>;
 def fdebug_prefix_map_EQ
----------------
probinson wrote:
> Probably should have HelpText, maybe something like this:
> "The default DWARF version to use, if a -g option causes DWARF debug info to 
> be produced"
> (it's probably not helpful to talk about overriding the target's default 
> version, I think)
Yep, I'd expect/be fine with all these options having something about DWARF in 
their long-form description.

@cmtice - would you mind switching this back to the original name & adding the 
relevant help text?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69822/new/

https://reviews.llvm.org/D69822



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to