[Mesa-dev] [PATCH 4/4] anv: Put everything about queries in genX_query.c
Jason Ekstrand
jason at jlekstrand.net
Mon Feb 20 18:46:08 UTC 2017
On Mon, Feb 20, 2017 at 10:35 AM, Emil Velikov <emil.l.velikov at gmail.com>
wrote:
> On 18 February 2017 at 23:59, Jason Ekstrand <jason at jlekstrand.net> wrote:
> > ---
> > src/intel/vulkan/Makefile.sources | 5 +-
> > src/intel/vulkan/anv_query.c | 171 -------------
> Won't this lead to this [albeit trivial] code built 4 times instead of
> once ? Worth mentioning in the commit message (build time should be
> the same afaict).
>
Yes it would and I did think about that. I briefly considered just picking
an arbitrary gen and only building the 3 common functions once but I
figured that was probably more hassle than it was worth.
> Regardless, I've verified that nothing sneaked in during the move and
> the makefile changes seem good.
> For 3 and 4:
> Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
>
> -Emil
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170220/ee90220f/attachment.html>
More information about the mesa-dev
mailing list