<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Sep 22, 2009, at 10:38 AM, Mike Stump wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Author: mrs<br>Date: Tue Sep 22 12:38:02 2009<br>New Revision: 82559<br><br>URL: <a href="http://llvm.org/viewvc/llvm-project?rev=82559&view=rev">http://llvm.org/viewvc/llvm-project?rev=82559&view=rev</a><br>Log:<br>Yes.<br><br></div></blockquote><div><br></div>Thanks for the clarification. The reason I added the FIXME was that the ABI spec say:</div><div><br></div><div>Identify <b>all virtual base classes, direct or indirect</b>, that are primary base classes for some other direct or indirect base class. Call these indirect primary base classes.</div><div><br></div><div>and the check made me believe that we were just looking for direct virtual base classes.</div><div><br></div><div>Thanks,</div><div>Anders</div><br></body></html>