Glenn Fowler wrote: > On Mon, 5 Feb 2007 15:42:49 -0500 James Carlson wrote: > > Glenn Fowler writes: > > That's exactly my question: rather than saying vaguely that it > > "screws-up" the compilation, what actual problems are seen? > > sorry, I don't like "x doesn't work", with no further info, either > > Roland, can you dig up the original msg for this problem?
The problem doesn't affect you directly. The issue is that we need a way to distingish between 32bit and 64bit because we merge arch/${platform}/inlcude/ast/ includes from 32bit and 64bit builds via "/usr/bin/diff -D<symbol>". The problem is that we can't use |_LP64| for <symbol> since the Sun Workshop/Forte/Studio compilers do not set |_LP64| (gcc set it, e.g. -- snip -- $ (touch foo.h; /usr/sfw/bin/cpp -m64 -dM foo.h) | fgrep _LP64 #define __LP64__ 1 #define _LP64 1 -- snip -- ) and "injecting" a |#include <sys/isa_defs.h>| will fork() the includes in a completely unpredictable manner (I've tried that - the sources compile and the code doesn't pass the test suite anymore (and I really don't have the time to figure out why it breaks somewhere in the ~~1000 source files we have... that's more a job for Mr. Sisyphus)). ---- Bye, Roland -- __ . . __ (o.\ \/ /.o) roland.mainz at nrubsig.org \__\/\/__/ MPEG specialist, C&&JAVA&&Sun&&Unix programmer /O /==\ O\ TEL +49 641 7950090 (;O/ \/ \O;)