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.