[OpenBIOS] [PATCH 10/10] Adding filll Forth primitive (v2)

Segher Boessenkool segher at kernel.crashing.org
Tue Aug 23 15:30:16 CEST 2011


>> This isn't going to work with -fstrict-aliasing, but I guess
>> openbios has that disabled anyway?

> I don't think it breaks strict aliasing rules. A cell is defined as a
> 32bit integer on 32bit architectures and this code will only be
> compiled on ppc32. I just wrote it as a uint32_t for clarity.

It breaks aliasing rules if whatever reads the data you write does not
read it as a type compatible with the type you wrote it as.  If  
everything
reads it as a 32-bit int, a cell, or a character type, it is fine; if  
not,
not.

Don't worry about it too much, I'm sure there are hundreds of other  
places
in openbios that do icky things like this.  It's in the nature of  
emulators/
interpreters :-)


Segher




More information about the OpenBIOS mailing list