Speedups/Cleanups: End of GSOC patch collection
Dimitrios Apostolou
jimis@gmx.net
Sat Aug 18 11:50:00 GMT 2012
2012-08-18 Dimitrios Apostolou <jimis@gmx.net>
* dwarf2out.c (output_indirect_string): Use
ASM_OUTPUT_INTERNAL_LABEL instead of slower ASM_OUTPUT_LABEL.
* varasm.c (assemble_string): Don't break string in chunks, this
is assembler specific and already done in most versions of
ASM_OUTPUT_ASCII.
I think there is no correctness issue regarding output_indirect_string()
since .debug_str are always compiler generated labels, and this gives a
small speedup with -g3 debug info.
And regarding assemble_string() I find it superfluous to break it in two
places. I found only the following versions of ASM_OUTPUT_ASCII not caring
about string length, I guess the assemblers don't have a limit:
arm.c: vmsdbgout.c:ASM_OUTPUT_ASCII
picochip.c: picochip_output_ascii()
pdp11.c: output_ascii()
Thanks,
Dimitris
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: varasm-assemble_string.diff
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20120818/f90e08f5/attachment.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: dwarf2out-indirect_string.diff
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20120818/f90e08f5/attachment-0001.ksh>
More information about the Gcc-patches
mailing list