This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] PR gas/16489: GAS doesn't complain about same register used for destination and index in AVX512 gather insn
- From: Michael Zolotukhin <michael dot v dot zolotukhin at gmail dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Jan Beulich <JBeulich at suse dot com>, Binutils <binutils at sourceware dot org>
- Date: Wed, 22 Jan 2014 21:13:45 +0400
- Subject: Re: [PATCH] PR gas/16489: GAS doesn't complain about same register used for destination and index in AVX512 gather insn
- Authentication-results: sourceware.org; auth=none
- References: <20140122140229 dot GA44568 at msticlxl57 dot ims dot intel dot com> <CAMe9rOqdG9MbZmgWscDNUwtxe4Qwmcgt3ppRR3u5bt9YAJaCmA at mail dot gmail dot com> <CANtU07-9W7yu1KrEL+TgnkYnxwv8GwMQR6M_q0TKuORFw-GCEA at mail dot gmail dot com> <CAMe9rOp3Z=pfy=RFL4ic5vJ4img=ZZAGDzfqf20z9S1GSw57Pg at mail dot gmail dot com> <CANtU07-0Jap2x_6mZRhc6t9T5zb5x8dBeha3=oPWPMEi66Pyug at mail dot gmail dot com> <CAMe9rOpDkKfXGnNh5rPQ0dJ2XE-Rd1h1bwKK12iZo2FiM3pOkA at mail dot gmail dot com>
> If the patch doesn't change anything in testsuite, it means it
> isn't necessary.
It is not necessary, but the assert allowing zmm-register is weaker
than it should be for AVX2 case, which doesn't allow zmm registers by
definition. The change just makes the assert correct, the way it
should've been before. But anyway, I could omit this change if
needed.
Michael