glapi: Don't emit remap data for individual extensions
authorIan Romanick <ian.d.romanick@intel.com>
Tue, 6 Sep 2011 20:02:13 +0000 (13:02 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Mon, 19 Sep 2011 19:02:21 +0000 (12:02 -0700)
All of the extensions actually supported by Mesa have been remapped by
remap.c for a long time.  Emitting all of these data structures is
just clutter.

Drivers that need additional functions remapped, should add
'offset="assign"' to the function definition in the .xml file.

The changes to remap_helper.h are in a follow-on ~8700 line patch that
would surely be rejected by the mailing list.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Chia-I Wu <olv@lunarg.com>
src/mapi/glapi/gen/remap_helper.py

index 367ae24c75cf62771371317792597757fc36dc47..0008634ff2e5f4662a8e3d0badd979672017c0f9 100644 (file)
@@ -74,7 +74,11 @@ class PrintGlRemap(gl_XML.gl_print_base):
                pool_indices = {}
 
                print '/* this is internal to remap.c */'
-               print '#ifdef need_MESA_remap_table'
+               print '#ifndef need_MESA_remap_table'
+               print '#error Only remap.c should include this file!'
+               print '#endif /* need_MESA_remap_table */'
+               print ''
+
                print ''
                print 'static const char _mesa_function_pool[] ='
 
@@ -156,43 +160,6 @@ class PrintGlRemap(gl_XML.gl_print_base):
                print '   {    -1, -1 }'
                print '};'
                print ''
-
-               print '#endif /* need_MESA_remap_table */'
-               print ''
-
-               # output remap helpers for DRI drivers
-
-               for ext in extensions:
-                       funcs = []
-                       remapped = []
-                       for f in extension_functions[ext]:
-                               if f.assign_offset:
-                                       # these are handled above
-                                       remapped.append(f)
-                               else:
-                                       # these functions are either in the
-                                       # abi, or have offset -1
-                                       funcs.append(f)
-
-                       print '#if defined(need_%s)' % (ext)
-                       if remapped:
-                               print '/* functions defined in MESA_remap_table_functions are excluded */'
-
-                       # output extension functions that need to be mapped
-                       print 'static const struct gl_function_remap %s_functions[] = {' % (ext)
-                       for f in funcs:
-                               if f.offset >= 0:
-                                       print '   { %5d, _gloffset_%s },' \
-                                                       % (pool_indices[f], f.name)
-                               else:
-                                       print '   { %5d, -1 }, /* %s */' % \
-                                                       (pool_indices[f], f.name)
-                       print '   {    -1, -1 }'
-                       print '};'
-
-                       print '#endif'
-                       print ''
-
                return