[PATCH] D43387: [DebugInfo] Removed assert on missing CountVarDIE

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 16 11:23:31 PST 2018


aprantl added a comment.

Can you add a testcase by, e.g, manually removing the dbg.value for the count from an existing testcase?



================
Comment at: lib/CodeGen/AsmPrinter/DwarfUnit.cpp:1388
+    else
+      ; // Should this case be handled?
   } else if (Count != -1)
----------------
That else statement was more meant as a question for you :-)



https://reviews.llvm.org/D43387





More information about the llvm-commits mailing list