[Bug 1607] __RTEMS_SIZEOF_VOID_P__ flawed design

Fri Feb 25 06:02:33 UTC 2011

https://www.rtems.org/bugzilla/show_bug.cgi?id=1607

--- Comment #31 from Ralf Corsepius <ralf.corsepius at rtems.org> 2011-02-25 00:02:31 CST ---
(In reply to comment #0)
> This change is a massive design flaw:
> 
> > 2010-06-29 Joel Sherrill <joel.sherrill at oarcorp.com>
> > 
> > * configure.ac, score/include/rtems/score/percpu.h: Add
> > __RTEMS_SIZEOF_VOID_P__ to cpuopts.h so percpu.h has this information
> > available during build and after installation.
> 
> Sizes are compile time constants and therefore may depend upon compiler 
> flags (e.g. multilibs) flags. Defining them in a globally exported 
> header being shared by different architecture variants qualifies is absurd.
> 
> Proposal: Joel to revert his whole __RTEMS_SIZEOF_VOID_P__ patch-set.

FYI:
m32c-rtems4.11/m32cm/config.h:#define SIZEOF_VOIDP 4  
m32c-rtems4.11/config.h:#define SIZEOF_VOIDP 2

=> This design flaw is not hypothetical, it is for real.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.