<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Oct 18, 2018 at 3:17 PM Jan Vesely <<a href="mailto:jan.vesely@rutgers.edu">jan.vesely@rutgers.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I think this patch breaks the build:<br>
<a href="https://travis-ci.org/jvesely/mesa/jobs/443356781" rel="noreferrer" target="_blank">https://travis-ci.org/jvesely/mesa/jobs/443356781</a><br></blockquote><div><br></div><div>Interesting.  It appears that it breaks when you have one of the WSI extensions disabled.  I'll take a look.</div><div><br></div><div>--Jason</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Jan<br>
<br>
On Tue, 2018-10-16 at 08:18 -0500, Jason Ekstrand wrote:<br>
> FYI, patch 1 is required for this patch to build.  It also means this patch <br>
> found a nice little bug.  I'll respond to patch 1 in more detail after the <br>
> SI call tomorrow.<br>
> <br>
> --Jason<br>
> <br>
> <br>
> On October 16, 2018 06:49:35 Lionel Landwerlin <br>
> <<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a>> wrote:<br>
> <br>
> > Reviewed-by: Lionel Landwerlin <<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a>><br>
> > <br>
> > On 15/10/2018 04:47, Jason Ekstrand wrote:<br>
> > > We don't need weak references to instance entrypoints because we never<br>
> > > have more than one of each so we don't need the NULL fall-back.  This<br>
> > > also helps us avoid forgetting things because we now get link errors for<br>
> > > missing instance entrypoints.<br>
> > > ---<br>
> > >   src/intel/vulkan/anv_entrypoints_gen.py | 13 -------------<br>
> > >   1 file changed, 13 deletions(-)<br>
> > > <br>
> > > diff --git a/src/intel/vulkan/anv_entrypoints_gen.py <br>
> > > b/src/intel/vulkan/anv_entrypoints_gen.py<br>
> > > index beb658b8660..25a532fd706 100644<br>
> > > --- a/src/intel/vulkan/anv_entrypoints_gen.py<br>
> > > +++ b/src/intel/vulkan/anv_entrypoints_gen.py<br>
> > > @@ -227,19 +227,6 @@ ${strmap(device_strmap, 'device')}<br>
> > >    * either pick the correct entry point.<br>
> > >    */<br>
> > > <br>
> > > -% for e in instance_entrypoints:<br>
> > > -  % if e.alias:<br>
> > > -    <% continue %><br>
> > > -  % endif<br>
> > > -  % if e.guard is not None:<br>
> > > -#ifdef ${e.guard}<br>
> > > -  % endif<br>
> > > -  ${e.return_type} ${e.prefixed_name('anv')}(${e.decl_params()}) <br>
> > > __attribute__ ((weak));<br>
> > > -  % if e.guard is not None:<br>
> > > -#endif // ${e.guard}<br>
> > > -  % endif<br>
> > > -% endfor<br>
> > > -<br>
> > >   const struct anv_instance_dispatch_table anv_instance_dispatch_table = {<br>
> > >   % for e in instance_entrypoints:<br>
> > >     % if e.guard is not None:<br>
> > <br>
> > <br>
> > _______________________________________________<br>
> > mesa-dev mailing list<br>
> > <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
> > <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
> <br>
> <br>
> <br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
<br>
</blockquote></div></div>