Hi,

Thank you very much for the answers. Can someone please commit this patch for 
me?

Thanks,
Sebastian

-----Original Message-----
From: Uros Bizjak [mailto:ubiz...@gmail.com] 
Sent: Wednesday, May 24, 2017 3:31 PM
To: Sandra Loosemore <san...@codesourcery.com>
Cc: Peryt, Sebastian <sebastian.pe...@intel.com>; gcc-patches@gcc.gnu.org; 
Koval, Julia <julia.ko...@intel.com>; kirill.yuk...@gmail.com
Subject: Re: [PATCH] Match x86 family machine constraints section with 
constarints.md

On Tue, May 23, 2017 at 5:33 PM, Sandra Loosemore <san...@codesourcery.com> 
wrote:
> On 04/28/2017 03:30 AM, Peryt, Sebastian wrote:
>>
>> Hi,
>>
>> Thank you for your comments. I edited my patch accordingly. As for 
>> some of your doubts:
>> - REX is  the opcode prefix to access 64-bit register extensions 
>> introduced in IA-32e mode.
>> - EVEX is the encoding prefix which applies to SIMD operating 
>> instructions operating on XMM, YMM and ZMM registers. It was 
>> introduced with AVX-512 instructions.
>> - "number factor of four" that means that sources start in a multiple 
>> of 4 boundary. This is used for some of instructions.
>>
>> Also I'd like to add that this whole patch is strictly based on 
>> docstring parts of constraints that are present in 
>> config/i386/constraints.md but not in documentation (md.texi file). 
>> There is no new (new as in nonexistent in
>> code) content.
>>
>> I'm also adding Kirill Yukhin to CC, because I believe he is the 
>> correct person that can catch any technical errors if any has slipped-in.
>
>
> The grammar/markup/etc are OK now, but I can't comment on technical 
> correctness of the information.

LGTM.

Thanks,
Uros.

Reply via email to