Hi Bill,<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 12, 2012 at 2:01 PM, Bill Wendling <span dir="ltr"><<a href="mailto:isanbard@gmail.com" target="_blank">isanbard@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: void<br>
Date: Mon Nov 12 16:01:56 2012<br>
New Revision: 167761<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=167761&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=167761&view=rev</a><br>
Log:<br>
Don't test for incomplete types.<br>
<br>
Modified:<br>
    cfe/trunk/lib/Sema/SemaStmtAsm.cpp<br>
    cfe/trunk/test/CodeGen/x86_32-inline-asm.c<br></blockquote><div><br></div><div> I've temporarily reverted this for you since it seems like it was causing test failures in ToT (SemaTemplate/instantiate-expr-1.cpp). Also, not quite sure what you're trying to do with the fix? If you'd like people to be able to load from incomplete types (likely use case) you'll need to avoid the validation code. Is it erroring out here? Also, that testcase looks kinda funky, it'd be nice to get a different testcase when you reapply than the first one.</div>
<div><br></div><div>-eric</div></div></div>