anv/entrypoints: Only generate entrypoints for supported features
authorJason Ekstrand <jason.ekstrand@intel.com>
Tue, 14 Feb 2017 18:24:14 +0000 (10:24 -0800)
committerJason Ekstrand <jason.ekstrand@intel.com>
Tue, 14 Feb 2017 22:18:55 +0000 (14:18 -0800)
This changes the way anv_entrypoints_gen.py works from generating a
table containing every single entrypoint in the XML to just the ones
that we actually need.  There's no reason for us to burn entrypoint
table space on a bunch of NV extensions we never plan to implement.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
src/intel/vulkan/anv_entrypoints_gen.py

index 0eb523d7ac9ba314ab16fe710c402a52b3377585..93511ec95e445ba6837bc27ad5e643d93bdde1c9 100644 (file)
 import sys
 import xml.etree.ElementTree as ET
 
+max_api_version = 1.0
+
+supported_extensions = [
+   'VK_KHR_get_physical_device_properties2',
+   'VK_KHR_maintenance1',
+   'VK_KHR_sampler_mirror_clamp_to_edge',
+   'VK_KHR_shader_draw_parameters',
+   'VK_KHR_surface',
+   'VK_KHR_swapchain',
+   'VK_KHR_wayland_surface',
+   'VK_KHR_xcb_surface',
+   'VK_KHR_xlib_surface',
+]
+
 # We generate a static hash table for entry point lookup
 # (vkGetProcAddress). We use a linear congruential generator for our hash
 # function and a power-of-two size table. The prime numbers are determined
@@ -66,10 +80,32 @@ elif (sys.argv[1] == "code"):
 # Extract the entry points from the registry
 def get_entrypoints(doc, entrypoints_to_defines):
     entrypoints = []
-    commands = doc.findall('./commands/command')
-    for i, command in enumerate(commands):
+
+    enabled_commands = set()
+    for feature in doc.findall('./feature'):
+        assert feature.attrib['api'] == 'vulkan'
+        if float(feature.attrib['number']) > max_api_version:
+            continue
+
+        for command in feature.findall('./require/command'):
+            enabled_commands.add(command.attrib['name'])
+
+    for extension in doc.findall('.extensions/extension'):
+        if extension.attrib['name'] not in supported_extensions:
+            continue
+
+        assert extension.attrib['supported'] == 'vulkan'
+        for command in extension.findall('./require/command'):
+            enabled_commands.add(command.attrib['name'])
+
+    index = 0
+    for command in doc.findall('./commands/command'):
         type = command.find('./proto/type').text
         fullname = command.find('./proto/name').text
+
+        if fullname not in enabled_commands:
+            continue
+
         shortname = fullname[2:]
         params = map(lambda p: "".join(p.itertext()), command.findall('./param'))
         params = ', '.join(params)
@@ -77,7 +113,9 @@ def get_entrypoints(doc, entrypoints_to_defines):
             guard = entrypoints_to_defines[fullname]
         else:
             guard = None
-        entrypoints.append((type, shortname, params, i, hash(fullname), guard))
+        entrypoints.append((type, shortname, params, index, hash(fullname), guard))
+        index += 1
+
     return entrypoints
 
 # Maps entry points to extension defines