Re: PATCH: [3.3 branch] IA64 bootstrap failure

["H. J. Lu" <hjl at lucon dot org>]

On Tue, Jul 08, 2003 at 10:02:13PM -0700, Zack Weinberg wrote:
> "H. J. Lu" <hjl@lucon.org> writes:
> 
> > On Tue, Jul 08, 2003 at 11:19:13PM -0400, Hans-Peter Nilsson wrote:
> >> On Tue, 8 Jul 2003, H. J. Lu wrote:
> >> > I am testing this patch now.
> >> 
> >> +int x [2] =
> >> +{
> >> +  __builtin_constant_p (10) ? 10 : -1,
> >> 
> >> All other issues aside, this part doesn't cause the intended
> >> failure if __builtin_constant_p falsely returns false; it'll
> >> just set that element to -1.  You want two arrays or a
> >> two-dimensional array, with those (negative or variable-indexed)
> >> expressions in the []-parts.
> >> 
> >
> > This patch should be conservative enough.
> 
> What part of "FIX THE REAL BUG" don't you understand?
> 

For that particular problem, the file is NOT miscompiled by the stage2
compiler. It is just different from the one compiled by the stage1
compiler. Since stage1 and stage2 have different source files, see
the diffs enclosed here, it doesn't make much senses to compare outputs
from those 2 compilers. Unless there is a difference between stage1
and stage2 of the same source files, such a comparison doesn't mean
much.


H.J.

Attachment: diffs.bz2
Description: BZip2 compressed data