[PATCH] D139989: [clang][DebugInfo] Add DW_AT_default_value support for template template parameters

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 11:17:02 PST 2022


dblaikie added inline comments.


================
Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:2095-2097
+        NamedDecl const *ND = Args.TList->getParam(i);
+        defaultParameter = TemplateUtils::isSubstitutedDefaultArgument(
+            CGM.getContext(), TA, ND, Args.Args, Args.TList->getDepth());
----------------
Could we pull this snipped out and do it before the switch - so we don't have to repeat it for each different kind of template parameter type?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139989



More information about the llvm-commits mailing list