Looks great. <br><br>Minor nits:<br><br>* LLVM coding style uses 'SomeClass *A' instead of 'SomeClass* A'.<br><br>* +  // For C++ constructor add initializers to CFG.<br>+  if (const CXXConstructorDecl *CD = dyn_cast_or_null<CXXConstructorDecl>(D)) {<br>
+    for (CXXConstructorDecl::init_const_reverse_iterator I = CD->init_rbegin()<br>+        , E = CD->init_rend(); I != E; ++I) {<br><br>Please put the comma at the end of the line.<br><br>+      B = addInitializer(*I);<br>
+      if (badCFG)<br>+        return NULL;<br>+    }<br>+  }<br><br><div class="gmail_quote">2010/10/4 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 support for C++ initializers in constructors. The output is similar to the one generated for VarDecls. I've also added two simple test cases.<div>
<br></div><div>Please approve for commit.</div><div><br></div><font color="#888888">
<div>Marcin</div>
</font></blockquote></div><br>