r217994 - [X86, inline-asm] Check that the input size is correct for constraints R, q, Q,

Hans Wennborg hans at chromium.org
Thu Sep 18 13:04:41 PDT 2014


On Thu, Sep 18, 2014 at 11:34 AM, Eric Christopher <echristo at gmail.com> wrote:
>>
>
>
>> +  case 'x':
>> +  case 'f':
>> +  case 't':
>> +  case 'u':
>> +    return Size <= 128;
>
>
> Hans pointed out that you'll have problems here with _m256 and wanting an
> avx register. There is, afaict, no separate constraint for "gimme an avx
> register" that's different from SSE registers. There is the 'v' constraint
> which works for avx 512 (evex encoded) registers.

How about the attached patch?

 - Hans
-------------- next part --------------
A non-text attachment was scrubbed...
Name: constraints.patch
Type: application/octet-stream
Size: 1677 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140918/f0394fbf/attachment.obj>


More information about the cfe-commits mailing list