Re: PATCH: PR target/39911: The 'z' suffix doesn't work with 16bit integer insn

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Mon, Apr 27, 2009 at 1:26 PM, Uros Bizjak <ubizjak@gmail.com> wrote:
> H.J. Lu wrote:
>
>> I checked the history on i386.c. %z works on x87 insns as far as back
>> to the initial version on gcc.gnu.org in 1992 and %z never worked on
>> memory operand with integer insns. I don't believe we should break
>> %z now. We should use %Z to support integer insns.
>>
>
> The problem is, that users want "%z" due to the comment in i386.md. Whether
> gcc uses "%Z" or "%z" internally does not matter at all. I propose that we
> go ahead with my patch to see if/what breaks. It is just a matter of
> changing the letter throughout a couple of source files to switch the
> modifiers.
>

Since we won't change 4.3/4.4 branches, we should update comments
on 4.3/4.4 branches to reflect how 'z' is actually used in gcc. It should have
zero impact.. OK for 4.3/4.4?

Thanks.

-- 
H.J.
---
2009-04-27  H.J. Lu  <hongjiu.lu@intel.com>

	* config/i386/i386.c (print_operand): Update comments for 'z'.
	* config/i386/i386.md: Likewise.

Attachment: gcc-z-1.patch
Description: Text document