<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Feb 14, 2017 at 11:34 AM, Chad Versace <span dir="ltr"><<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue 14 Feb 2017, Matt Turner wrote:<br>
> On Tue, Feb 14, 2017 at 11:21 AM, Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> wrote:<br>
> > On Tue, Feb 14, 2017 at 11:18 AM, Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> wrote:<br>
> >> On Tue, Feb 14, 2017 at 10:59 AM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> >>> I'm not sure how I feel about the silent fall-backs.  At least in the Vulkan<br>
> >>> driver, we should fail to compile if we can't get build-id.  Otherwise,<br>
> >>> you'll end up compiling a driver that will always fail device creation.<br>
> >><br>
> >> That was really an attempt to preempt questions about Windows.<br>
> >><br>
> >> I am happy to drop it.<br>
> ><br>
> > Think-o. No, it's necessary for systems that don't have<br>
> > dl_iterate_phdr (Windows, AFAIK).<br>
><br>
> I promise I'll stop replying to myself after this...<br>
><br>
> I guess dropping the fallback and simply wrapping build_id.c in<br>
> HAVE_DL_ITERATE_PHDR is sufficient? Windows will build an empty source<br>
> file, which is fine because no code should ever attempt to use it on<br>
> Windows; and there would be no chance of the Vulkan driver calling<br>
> (non-existent) fallback code.<br>
<br>
</div></div>I'm in favor of dropping the fallback stubs.<br>
</blockquote></div><br></div><div class="gmail_extra">I don't really care too much how it works so long as anv fails to compile.  I could see the stubs maybe being useful if you wanted to have the shader cache stuff silently and automatically fall back to disabled.  Or, the shader cache stuff could have a #if.  It doesn't matter to me.<br></div></div>