From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 65134 invoked by alias); 14 Feb 2020 14:23:29 -0000 Mailing-List: contact binutils-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: binutils-owner@sourceware.org Received: (qmail 65126 invoked by uid 89); 14 Feb 2020 14:23:28 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-4.5 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.1 spammy=H*f:sk:fySYpQL, H*MI:sk:fySYpQL, H*i:sk:fySYpQL X-HELO: mx2.suse.de Received: from mx2.suse.de (HELO mx2.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 14 Feb 2020 14:23:27 +0000 Received: from relay2.suse.de (unknown [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id AF53BB396; Fri, 14 Feb 2020 14:23:25 +0000 (UTC) Subject: Re: [PATCH v8 2/2] x86/AT&T: don't default to byte source for ambiguous for MOVSX/MOVZX To: "H.J. Lu" Cc: "binutils@sourceware.org" References: <92fd2cf8-90a0-98f8-9453-c49d911daec6@suse.com> <84508928-4fdb-997f-b187-66ea614fd9bc@suse.com> From: Jan Beulich Message-ID: Date: Fri, 14 Feb 2020 14:23:00 -0000 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.4.2 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2020-02/txt/msg00359.txt.bz2 On 14.02.2020 15:16, H.J. Lu wrote: > On Fri, Feb 14, 2020 at 5:54 AM Jan Beulich wrote: >> >> On 14.02.2020 13:28, H.J. Lu wrote: >>> On Fri, Feb 14, 2020 at 4:26 AM Jan Beulich wrote: >>>> >>>> As pointed out in "x86: replace adhoc (partly wrong) ambiguous operand >>>> checking for MOVSX/MOVZX" silently guessing what the programmer may have >>>> meant is not helpful, the more that we don't do so elsewhere anymore >>>> (except in cases where it is overwhelmingly likely that the other case >>>> isn't meant, like here for it meant to be a "sign/zero extension" from >>>> 16 bits to 16 bits). >>>> >>>> gas/ >>>> 2020-02-XX Jan Beulich >>>> >>>> PR gas/25438 >>>> * config/tc-i386.c (process_suffix): Default movsx/movzx to byte >>>> suffix only when destination is a word reg. >>>> testsuite/gas/i386/noreg16.l, testsuite/gas/i386/noreg32.l, >>>> testsuite/gas/i386/noreg64.l: Adjust expectations. >>> >>> No need for this since this is documented behavior of AT&T syntax. >> >> I've just looked at the documentation again: As said in the >> other reply to your doc change, these mnemonics aren't >> mentioned as legal in Solaris'es AT&T spec. And I also >> can't find gas doc saying so. Would you please point me at >> where this is being documented? > > Solaris spec doesn't mention movsx[bwl] nor movsx. Right, so where did you take from that "this is documented behavior"? Jan