<br><br><div class="gmail_quote">On Fri, Aug 28, 2009 at 02:52, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Thu, Aug 27, 2009 at 5:31 PM, Anders Johnsen<<a href="mailto:skabet@gmail.com">skabet@gmail.com</a>> wrote:<br>
> Hi,<br>
> This is an attempt to implement instantiation of constructor initializers,<br>
> when the class is dependent.<br>
<br>
</div>+    // Assign all the initializers to the new constructor.<br>
+    ActOnMemInitializers(DeclPtrTy::make(Function), PointOfInstantiation,<br>
+        (MemInitTy**)NewItems.data(), NewItems.size());<br>
<br>
The cast here (and in general, casting to an Object**) looks<br>
suspicious; it often leads to a strict-aliasing violation.</blockquote><div><br></div><div>Yes, I've fixed this. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
   Stmt *Pattern = 0;<br>
   if (PatternDecl)<br>
-    Pattern = PatternDecl->getBody(PatternDecl);<br>
+    Pattern = PatternDecl->getBody((const FunctionDecl*&)PatternDecl);<br>
<br>
   if (!Pattern)<br>
     return;<br>
<br>
Similarly here.<br>
</blockquote><div>also fixed (was not required) </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><br>
It would be easier to review this if the addition of RParenLoc were<br>
done in a separate patch.<br>
<font color="#888888"></font></blockquote><div>Okay, I've split it.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><font color="#888888"><br>
-Eli<br>
</font></blockquote></div>Thank you for the review!