<p dir="ltr">OK, I agree.</p>
<p dir="ltr">Can I commit without an extra review iteration once I address these issues?</p>
<div class="gmail_quote">26.02.2013 21:31 пользователь "John McCall" <<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>> написал:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Feb 26, 2013, at 7:48 AM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br>
> That's a great suggestion!<br>
><br>
> Attached is a patch that does as you've described and also addresses<br>
> all the TODOs I've left for myself in the previous patch.<br>
<br>
I like how this is shaping up!  Just a few comments:<br>
<br>
This:<br>
  getContext().getTargetInfo()<br>
is just CGM.getTarget().<br>
<br>
+  llvm::BasicBlock *CallVbaseCtorsBB = CGF.createBasicBlock("ctor.init_vbases"),<br>
+                   *SkipVbaseCtorsBB = CGF.createBasicBlock("ctor.skip_vbases");<br>
<br>
Prefer separate declarations if you can't fit them on the same line.<br>
<br>
> One questionable change is the removal of "static" for GetVTTParameter.<br>
> I didn't want to make it a CGF method as it should be used anywhere<br>
> except ItaniumCXXABI.cpp but we have to use it in CGClass.cpp until<br>
> the full abstraction is finished.<br>
> Please tell me if I cut a corner too much there and how to do it better.<br>
<br>
Go ahead and move it to CodeGenFunction for now.<br>
<br>
John.<br>
</blockquote></div>