Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

[Joern Rennecke <joern dot rennecke at embecosm dot com>]

Quoting "Joseph S. Myers" <joseph@codesourcery.com>:

> The c-opts.c change to include tm.h seems independent of the rest of the
> patch, and is incorrect since the file already has an explicit tm.h
> include with a comment saying why it's included.

Yes, I've left this bit out now.  It used to be necessary, but it has been
obsoleted by your c-opts.c patch last month.

> I think it would be a
> good idea for the java/expr.c include (which I can't approve, Java patches
> should best be CC:ed to java-patches) to have a comment saying what target
> macros are used

Added.

> Similar comments also apply to the fortran/trans-types.c change: send to
> the fortran list as well as gcc-patches, include a comment listing the
> target macros involved (BOOL_TYPE_SIZE CHAR_TYPE_SIZE DOUBLE_TYPE_SIZE
> FLOAT_TYPE_SIZE INT_TYPE_SIZE LIBGCC2_HAS_TF_MODE LONG_DOUBLE_TYPE_SIZE
> LONG_LONG_TYPE_SIZE LONG_TYPE_SIZE POINTER_SIZE SHORT_TYPE_SIZE
> SIZE_TYPE).

Instead of including SIZE_TYPE in this list, I've removed the #if 0 code
that did (not) use it.
I've also added 2011 to the Copyright years in touched files that  
didn't already
have it.

Attachment: pr46500-patch-20110507-2.gz
Description: GNU Zip compressed data