[Mesa-dev] [PATCH 2/3] anv: Stop generating weak references for instance entrypoints

Lionel Landwerlin lionel.g.landwerlin at intel.com
Tue Oct 16 11:49:30 UTC 2018


Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at 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:




More information about the mesa-dev mailing list