re: CVS commit: src/sys/arch/atari/stand/installboot

[matthew green <mrg%eterna.com.au@localhost>]

Nick Hudson writes:
> On 11/24/14 07:52, Izumi Tsutsui wrote:
> > Module Name:	src
> > Committed By:	tsutsui
> > Date:		Mon Nov 24 07:52:04 UTC 2014
> >
> > Modified Files:
> > 	src/sys/arch/atari/stand/installboot: Makefile
> >
> > Log Message:
> > Specify -fno-strict-aliasing as a temporary workaround for gcc48.
> >
> > The existing abcksum() also violates strict-aliasing rule
> > (while current gcc48 doesn't warn it) and fixing all violations
> > strictly requires whole reorganization of boot sector structures.
> > But it won't happen soon and this MD installboot should be integrated
> > into MI installboot(8) in future, and it requires whole overhaul anyway.
> > See long discussion in source-changes-d@ for details.
> >
> > Should be pulled up to netbsd-7 if we switches m68k to using gcc48.
> >
> Why not pullup in anticipation?

infact, given the strict aliasing violations in this file, there
is no guarantee that GCC 4.5 isn't miscompiling it already just
without the additional warnings in 4.8...

ie, this seems sane regardless of what happens.  same for -6?


.mrg.