FYI, patch 1 is required for this patch to build. It also means this patch
found a nice little bug. I'll respond to patch 1 in more detail after the
SI call tomorrow.
--Jason
On October 16, 2018 06:49:35 Lionel Landwerlin
<lionel.g.landwer...@intel.com> wrote:
Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>
On 15/10/2018 04:47, Jason Ekstrand wrote:
We don't need weak references to instance entrypoints because we never
have more than one of each so we don't need the NULL fall-back. This
also helps us avoid forgetting things because we now get link errors for
missing instance entrypoints.
---
src/intel/vulkan/anv_entrypoints_gen.py | 13 -------------
1 file changed, 13 deletions(-)
diff --git a/src/intel/vulkan/anv_entrypoints_gen.py
b/src/intel/vulkan/anv_entrypoints_gen.py
index beb658b8660..25a532fd706 100644
--- a/src/intel/vulkan/anv_entrypoints_gen.py
+++ b/src/intel/vulkan/anv_entrypoints_gen.py
@@ -227,19 +227,6 @@ ${strmap(device_strmap, 'device')}
* either pick the correct entry point.
*/
-% for e in instance_entrypoints:
- % if e.alias:
- <% continue %>
- % endif
- % if e.guard is not None:
-#ifdef ${e.guard}
- % endif
- ${e.return_type} ${e.prefixed_name('anv')}(${e.decl_params()})
__attribute__ ((weak));
- % if e.guard is not None:
-#endif // ${e.guard}
- % endif
-% endfor
-
const struct anv_instance_dispatch_table anv_instance_dispatch_table = {
% for e in instance_entrypoints:
% if e.guard is not None:
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
