diff options
author | Jason Ekstrand <[email protected]> | 2017-02-14 10:24:14 -0800 |
---|---|---|
committer | Jason Ekstrand <[email protected]> | 2017-02-14 14:18:55 -0800 |
commit | 114c281e707919a1420d3a20219ce4392d8b360d (patch) | |
tree | f960efa0993b81a7234afb333d2bb0bd58523aba | |
parent | 6319bfc2a6497d708ead536b9a6d5d5a00c1f2f3 (diff) |
anv/entrypoints: Only generate entrypoints for supported features
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 <[email protected]>
-rw-r--r-- | src/intel/vulkan/anv_entrypoints_gen.py | 44 |
1 files changed, 41 insertions, 3 deletions
diff --git a/src/intel/vulkan/anv_entrypoints_gen.py b/src/intel/vulkan/anv_entrypoints_gen.py index 0eb523d7ac9..93511ec95e4 100644 --- a/src/intel/vulkan/anv_entrypoints_gen.py +++ b/src/intel/vulkan/anv_entrypoints_gen.py @@ -25,6 +25,20 @@ 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 |