Bug 4021

Summary: slang_execute.c doesn't build on 64-bit CPU's
Product: Mesa Reporter: Alan Coopersmith <alan.coopersmith>
Component: Mesa coreAssignee: mesa-dev
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: high    
Version: git   
Hardware: All   
OS: All   
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 1690    

Description Alan Coopersmith 2005-08-09 07:09:39 UTC
Mesa/src/mesa/shader/slang/slang_execute.c contains asserts in the 
_slang_execute function to prevent building 64-bit binaries:

        /* assume 32-bit machine */
        static_assert(sizeof (GLfloat) == 4);
        static_assert(sizeof (GLfloat *) == 4);
        static_assert(sizeof (GLuint) == 4);
        static_assert(sizeof (GLuint *) == 4);

(gcc won't actually issue errors for these asserts since the static_assert
macro translates into code that gcc doesn't think is an error, but many other
compilers do - that's a separate bug I'll report in a minute.)
Comment 1 Brian Paul 2005-08-11 00:40:35 UTC
I've disabled the assertions for now.  I don't see why they're needed.  The
shading language stuff isn't finished yet and isn't enabled anyway.
Comment 2 Adam Jackson 2009-08-24 12:23:20 UTC
Mass version move, cvs -> git

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.