This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: warning patch for gcc/config files
- To: rth at cygnus dot com
- Subject: Re: warning patch for gcc/config files
- From: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- Date: Fri, 23 Oct 1998 15:50:50 -0400 (EDT)
- Cc: davem at dm dot cobaltmicro dot com, egcs-patches at cygnus dot com
> From: Richard Henderson <rth@cygnus.com>
>
> On Fri, Oct 23, 1998 at 10:51:02AM -0400, Kaveh R. Ghazi wrote:
> > Regarding Jeff's conditional approval below, would you please
> > let me know if there are any objectionable items in
> > http://www.cygnus.com/ml/egcs-patches/1998-Oct/0638.html with respect to
> > the bits you each maintain?
>
> The Alpha bits are fine.
Thanks, I've installed them.
> I do have one question as to why
>
> -#define Pmode (TARGET_PTR64 ? DImode : SImode)
> +#define Pmode ((enum machine_mode)(TARGET_PTR64 ? DImode : SImode))
>
> is necessary. The two arms of the ?: have the same type, which is
> passed on to the result.
> r~
>
> [...]
>
> I'd like you to delay this change, as I believe that it points out
> a bug in gcc.
> r~
Sure, I'll hold off on the Pmode definition casts.
Would you like to comment on the other sparc patches, or wait for
davem to do it?
--Kaveh
--
Kaveh R. Ghazi Engagement Manager / Project Services
ghazi@caip.rutgers.edu Icon CMT Corp.