Michael137 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());
----------------
dblaikie wrote:
> 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?
yup makes sense!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139989

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

Reply via email to