Re: [patch][PCH] Do not write/read asm_out_file

[Steven Bosscher <stevenb dot gcc at gmail dot com>]

On Wed, Jun 6, 2012 at 10:45 AM, Richard Guenther
<richard.guenther@gmail.com> wrote:
> Please split up the patch into one dealing with #ident exclusively and
> a followup with the other pieces.

Hello,

This is the patch to deal with #ident. It removes the ASM_OUTPUT_IDENT
and IDENT_ASM_OP target macros, and replaces them with a single target
hook. For targets that treat front-end #idents different from the GCC
version string .ident, the hooks can use cgraph_status to distinguish
them.

This patch was bootstrapped and tested on powerpc64-unknown-linux-gnu
with all languages enabled (c,c++,objc,obj-c++,fortran,java,ada), and
on x86_64-unknown-linux-gnu (c,c++,objc,fortran,ada). In addition, I
have built cross-tools in combined trees (src+gcc) from
powerpc64-unknown-linux-gnu to cris-elf, mips-elf, rx-elf, and
microblaze-elf, and verified that the differences in assembler output
are negligible (only #APP / #NO_APP, which is emitted for all
top-level asms, and an extra blank line). I have also tested mips-sim
and compared the test results without and with patch.

OK for trunk?

Ciao!
Steven

Attachment: fe_no_output_h_ident_v1.diff.txt
Description: Text document