<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 18, 2014 at 8:17 AM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2014-04-18 1:38 GMT+04:00 Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>>:<br>
<div class="">> ================<br>
> Comment at: test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp:459<br>
> @@ +458,3 @@<br>
> +namespace Test12 {<br>
> +struct X : B, A { };<br>
> +<br>
> ----------------<br>
> Going forward, I'd really prefer it if our inheritance hierarchy tests could all be standalone.  See the microsoft-abi-vbtables.cpp test cases, where the contents of any namespace can be cut-and-paste into a standalone .cpp file and compiled.<br>

><br>
> In this test, I have to go lookup ::A and ::B in the right namespace to figure out what kind of record this is going to be.  It's worth a few lines of duplicated test code to avoid that.<br>
<br>
</div>Like r206614?<br>
</blockquote></div><br></div><div class="gmail_extra">Yep, thanks!</div></div>