On Sun, Jul 23, 2017 at 5:10 AM, David Edelsohn <dje....@gmail.com> wrote:
> dw2_asm_output_nstring() essentially inlines the version of
> OUTPUT_ASCII for the GNU Assembler ELF file format for the debug_asm
> case to append the comment and calls OUTPUT_ASCII when not debugging.
> The DWARF portion of the GCC testsuite relies on the text that appears
> in the comments, so it needs to work somewhat on AIX.
>
> The function previously had been adjusted to emit the AIX assembler
> .byte directive.  The AIX assembler escapes double quotes with another
> double quote, not a backslash.  The AIX toolchain added support for
> the debug_macinfo section and macros with quotes generate assembler
> errors when debug_asm is enabled.
>
> This patch adjusts the function to emit a double quote instead of a
> backslash when XCOFF_DEBUGGING_INFO is defined for AIX.
>
> Bootstrapped on powerpc-ibm-aix7.2.0.0
>
> Okay?

Ok.

Richard.

>
> Thanks, David
>
> Index: dwarf2asm.c
> ===================================================================
> --- dwarf2asm.c (revision 250453)
> +++ dwarf2asm.c (working copy)
> @@ -345,7 +345,9 @@ dw2_asm_output_nstring (const char *str, size_t or
>        for (i = 0; i < len; i++)
>         {
>           int c = str[i];
> -         if (c == '\"' || c == '\\')
> +         if (c == '\"')
> +           fputc (XCOFF_DEBUGGING_INFO ? '\"' : '\\', asm_out_file);
> +         else if (c == '\\')
>             fputc ('\\', asm_out_file);
>           if (ISPRINT (c))
>             fputc (c, asm_out_file);

Reply via email to