<div dir="ltr">On Thu, Jun 6, 2013 at 4:02 PM, Tim Northover <span dir="ltr"><<a href="mailto:tnorthover@apple.com" target="_blank">tnorthover@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
The attached patch fixes an issue where Clang would try to create a bitcast from “<8 x i32>” to “x86_mmx” which has a different size. This caused an assertion failure on malformed inline assembly.<br>
<br>
The problem seems to come from the unexpected behaviour of the adjustInlineAsmType, and this patch alters it slightly to report incompatible cases and allow the CodeGen to issue a proper diagnostic.<br>
<br>
Is it OK to commit or have I done something wrong?<br><br></blockquote><div><br></div><div>Please use -verify instead of FileCheck to write tests for diagnostics.  Otherwise, looks fine.</div><div><br></div><div>-Eli</div>
</div></div></div>