[PATCH] Diagnose incompatible MMX asm constraints

Eli Friedman eli.friedman at gmail.com
Thu Jun 6 16:19:47 PDT 2013


On Thu, Jun 6, 2013 at 4:02 PM, Tim Northover <tnorthover at apple.com> wrote:

> Hi,
>
> 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.
>
> 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.
>
> Is it OK to commit or have I done something wrong?
>
>
Please use -verify instead of FileCheck to write tests for diagnostics.
 Otherwise, looks fine.

-Eli
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130606/16df4d81/attachment.html>


More information about the cfe-commits mailing list