This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug bootstrap/14462] [3.5 Regression] a-calend.adb:396:33: warning: value not in range of type "Ada.Calendar.Day_Duration"
- From: "dave at hiauly1 dot hia dot nrc dot ca" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 12 Mar 2004 15:46:08 -0000
- Subject: [Bug bootstrap/14462] [3.5 Regression] a-calend.adb:396:33: warning: value not in range of type "Ada.Calendar.Day_Duration"
- References: <20040306175828.14462.danglin@gcc.gnu.org>
- Reply-to: gcc-bugzilla at gcc dot gnu dot org
------- Additional Comments From dave at hiauly1 dot hia dot nrc dot ca 2004-03-12 15:46 -------
Subject: Re: [3.5 Regression] a-calend.adb:396:33: warni
> > 1) The ChangeLog entry for trans.c is not written using GCC standards.
> > Here it is:
> >
> > 2004-03-05 Richard Kenner <kenner@vlsi1.ultra.nyu.edu>
> >
> > * trans.c: Reflect GCC changes to fix bootstrap problem.
> > Add warning for suspicious aliasing unchecked conversion.
> >
> > ChangeLog entries are supposed to indicate the functions changed
> > and what was changed. They are *not* supposed to contain a justification
> > for the change. What the heck does "Reflect GCC changes" mean.
>
> We've found that the lack of a justification in changelogs in GCC in general
> makes it very hard to track failures and understand changes and is very time
> consuming, so we're certainly not willing to loose this very important extra
> piece of information.
Personally, I don't have a strong opinion on this issue as long as the
details of the actual change don't get lost. In the above, we don't
know which boostrap problem was fixed. I searched the gcc, gcc-bugs
and gcc-patches for a discussion on a bootstrap problem for Ada and
couldn't find anything.
If the change had said "Use RTX_INSN instead of 'i'", it would have been
immediately clear that what was being fixed was related to Paulo's change.
And as I noted, including a list of the changed functions/macros in the
log entry helps further to narrow the scope of the change. This detail
is important when problems such as this arise. Both Paulo and I thought
from your comments and the log entry that there was a second problem
that needed to be addressed.
With Jan's patch reverted, I can sucessfully bootstrap with Ada on
hppa-unknown-linux-gnu and hppa2.0w-hp-hpux11.00.
Dave
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14462