Re: [Mesa-dev] [PATCH 4/4] anv: Put everything about queries in genX_query.c

2017-02-20 Thread Jason Ekstrand
On Mon, Feb 20, 2017 at 10:35 AM, Emil Velikov wrote: > On 18 February 2017 at 23:59, Jason Ekstrand 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 o

Re: [Mesa-dev] [PATCH 4/4] anv: Put everything about queries in genX_query.c

2017-02-20 Thread Emil Velikov
On 18 February 2017 at 23:59, Jason Ekstrand 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

[Mesa-dev] [PATCH 4/4] anv: Put everything about queries in genX_query.c

2017-02-18 Thread Jason Ekstrand
--- src/intel/vulkan/Makefile.sources | 5 +- src/intel/vulkan/anv_query.c | 171 - src/intel/vulkan/genX_cmd_buffer.c | 303 --- src/intel/vulkan/genX_query.c | 477 + 4 files changed, 481 insertions(+), 475 deletio