On Mon, Apr 27, 2009 at 1:26 PM, Uros Bizjak 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 * config/i386/i386.c (print_operand): Update comments for 'z'. * config/i386/i386.md: Likewise.