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

Paul Robinson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 5 13:15:42 PST 2019


probinson added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:6171
+
+  if (DwarfVersion == 0)
+    DwarfVersion = ParseDwarfDefaultVersion(getToolChain(), Args);
----------------
MaskRay wrote:
>   lang=cpp
>   if (DwarfVersion == 0) {
>     DwarfVersion = ParseDwarfDefaultVersion(getToolChain(), Args);
>     if (DwarfVersion == 0)
>       DwarfVersion = getToolChain().GetDefaultDwarfVersion();
>   }
> 
> This part constructs a -cc1as command line. It is not covered by a test. Can you add one?
Hi @MaskRay , your version would have the unclaimed-option problem.


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

https://reviews.llvm.org/D69822





More information about the cfe-commits mailing list