Re: port-sparc64/46652 (sparc ofwboot.net fails to loadkernelviatftp)

[Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>]

The following reply was made to PR port-sparc64/46652; it has been noted by 
GNATS.

From: Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>
To: spz%NetBSD.org@localhost, mrg%eterna.com.au@localhost
Cc: gnats-bugs%NetBSD.org@localhost, tsutsui%ceres.dti.ne.jp@localhost
Subject: Re: port-sparc64/46652 (sparc ofwboot.net fails to loadkernelviatftp)
Date: Mon, 16 Jul 2012 11:47:23 +0900

 mrg@ wrote:
 
 > >  > I notice old gcc 4.1.3 complained about (int **) with WARNS=2:
 > >  > ---
 > >  > cc1: warnings being treated as errors
 > >  > /usr/src/sys/arch/sparc/stand/ofwboot/ofdev.c: In function 'devopen':
 > >  > /usr/src/sys/arch/sparc/stand/ofwboot/ofdev.c:539: warning: 
 > > dereferencing type-punned pointer will break strict-aliasing rules
 > >  > ---
 > >  > (probably that was the reason I changed it)
 > >  > 
 > >  > gcc 4.5.3 doesn't warn it so is it okay to assume it was a false alarm?
 > >  
 > >  gcc 4.5 seems to be smarter about it.
 > >  putting an int * into a location an int ** points to shouldn't cause 
 > > grief :)
 > >  
 > >  I would assume that the danger of someone building Sparc bootblocks on a
 > >  Vax is pretty low, too.
 
 Gcc version is specified by target or HAVE_GCC variable, not host,
 and I have not checked other compilers.
 
 > if necessary, i'd put a gcc 4.1-only Makefile conditional to disable
 > this warning or feature.
 
 Thanks.
 
 I'll apply the patch in PR with comment.
 
 ---
 Izumi Tsutsui