[PATCH] D69822: [clang] Add new -fdebug-default-version flag.

[David Blaikie via Phabricator via cfe-commits]

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