Understood, but I'll have to stick to "small" changes as I can't get the papers.
Uros pointed to: http://gcc.gnu.org/ml/gcc-patches/2010-05/msg00657.html which appears to just be *very* coincident timing. So I Rainer will fix it soon. I have a patch now based on that discussion. I used: case ';': if ((TARGET_MACHO || TARGET_SOLARIS) && ASSEMBLER_DIALECT == ASM_ATT) fputs (" ; ", file); else fputc (' ', file); return; though even better might be to assume 64bit implies more recent assembler: case ';': if ((TARGET_MACHO || TARGET_SOLARIS) && ASSEMBLER_DIALECT == ASM_ATT && !TARGET_64BIT) fputs (" ; ", file); else fputc (' ', file); return; I guess there was an obscure bug where -masm=intel didn't work for Darwin targets?. What, people use -masm=intel and masm/nasm/yasm instead of gas? Or just to human read the output? Thanks, - Jay ---------------------------------------- > From: ebotca...@adacore.com > To: jay.kr...@cornell.edu > Subject: Re: rep prefix doesn't work with Solaris 2.9 Sun assembler > Date: Tue, 11 May 2010 10:35:12 +0200 > CC: gcc@gcc.gnu.org > >> Proposed patch below/attached. >> (-w to hide indent change) > > See http://gcc.gnu.org/contribute.html for guidelines. > >> I'll open a bug. > > See http://gcc.gnu.org/bugs for guidelines. > > Generally speaking, posting a patch inlined in a message on gcc@gcc.gnu.org > will most likely result in it being lost and forgotten. In order to report > an issue, please open a ticket with bugzilla. In order to submit a patch, > please use gcc-patc...@gcc.gnu.org. In both cases, follow the guidelines > written down in the aforementioned documentation. Thanks in advance. > > -- > Eric Botcazou
