Hi Marcin,<br><br>Overall the patch looks good. But one problem:<br><br>@@ -126,7 +130,8 @@<br>                                                     idx::TranslationUnit *TU) {<br>   AnalysisContext *&AC = Contexts[D];<br>
   if (!AC)<br>-    AC = new AnalysisContext(D, TU, UseUnoptimizedCFG);<br>+    AC = new AnalysisContext(D, TU, UseUnoptimizedCFG, AddImplicitDtors,<br>+        AddInitializers);<br><br>Notice that the fourth parameter is bool addehedges = false. We could set it to false for now. In the future we would add an option for it, too. <br>
<br>You could spot this bug by actually running clang with the patch on a test case:<br><br>$ clang -cc1 -analyze -cfg-view -cfg-add-implicit-dtors scope.cpp<br><br>And see no effect with the new option.<br><br>And for each patch, you can run 'make test' to ensure it does not break regression tests.<br>
<br><br><div class="gmail_quote">2010/9/30 Marcin Świderski <span dir="ltr"><<a href="mailto:marcin.sfider@gmail.com">marcin.sfider@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Patch adds two new command line arguments:<div>-cfg-add-implicit-dtors - sets CFG::BuildOptions::AddImplicitDtors for AnalysisCosumer to true</div>-cfg-add-initializers - sets CFG::BuildOptions::AddInitializers for AnalysisCosumer to true<div>

<br></div><div>Please approve for commit.</div><div><br></div><font color="#888888"><div>Marcin</div>
</font></blockquote></div><br>