Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-16 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > On 30/04/16 09:52, Francisco Jerez wrote: >> Samuel Iglesias Gonsálvez writes: >> >>> Hello, >>> >>> This patch series continues adding arb_gpu_shader_fp64 support to the >>> Intel driver. Specifically, this targets the i965 scalar backend for >>> BDW+ hardw

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-16 Thread Samuel Iglesias Gonsálvez
On 30/04/16 09:52, Francisco Jerez wrote: > Samuel Iglesias Gonsálvez writes: > >> Hello, >> >> This patch series continues adding arb_gpu_shader_fp64 support to the >> Intel driver. Specifically, this targets the i965 scalar backend for >> BDW+ hardware (vec4 is still under research and gen7

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-10 Thread Samuel Iglesias Gonsálvez
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 03/05/16 20:59, Kenneth Graunke wrote: > Other than patches 37, 56, and ones you agreed to drop, the series > is: Reviewed-by: Kenneth Graunke > > I think you can go ahead and land all except those, and we can > land new solutions for those pr

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-04 Thread Samuel Iglesias Gonsálvez
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 04/05/16 21:28, Francisco Jerez wrote: > Samuel Iglesias Gonsálvez writes: > > > > On 03/05/16 20:59, Kenneth Graunke wrote: Other than patches 37, 56, and ones you agreed to drop, the series is: Reviewed-by: Kenneth Graunke

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-04 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA256 > > > > On 03/05/16 20:59, Kenneth Graunke wrote: >> Other than patches 37, 56, and ones you agreed to drop, the series >> is: Reviewed-by: Kenneth Graunke >> >> I think you can go ahead and land all except th

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-04 Thread Samuel Iglesias Gonsálvez
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 03/05/16 20:59, Kenneth Graunke wrote: > Other than patches 37, 56, and ones you agreed to drop, the series > is: Reviewed-by: Kenneth Graunke > > I think you can go ahead and land all except those, and we can > land new solutions for those pr

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-03 Thread Kenneth Graunke
Other than patches 37, 56, and ones you agreed to drop, the series is: Reviewed-by: Kenneth Graunke I think you can go ahead and land all except those, and we can land new solutions for those problems afterwards. We still need to fix the horiz_offset problem, and I think Curro's subscript() help

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-03 Thread Kenneth Graunke
On Tuesday, May 3, 2016 12:09:33 PM PDT Samuel Iglesias Gonsálvez wrote: > > On 02/05/16 23:50, Mark Janes wrote: > > Samuel Iglesias Gonsálvez writes: > > > >> Hello, > >> > >> This patch series continues adding arb_gpu_shader_fp64 support to > >> the Intel driver. Specifically, this targets the

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-03 Thread Samuel Iglesias Gonsálvez
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 02/05/16 23:50, Mark Janes wrote: > Samuel Iglesias Gonsálvez writes: > >> Hello, >> >> This patch series continues adding arb_gpu_shader_fp64 support to >> the Intel driver. Specifically, this targets the i965 scalar >> backend for BDW+ har

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-02 Thread Samuel Iglesias Gonsálvez
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 02/05/16 07:47, Jordan Justen wrote: > 7-10, 12-20, 36-43, 57-58: Reviewed-by: Jordan Justen > > > I also sent questions about 56 & 59. > Thanks a lot for your reviews! I am still catching up email and doing the required changes to each pat

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-02 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > Hello, > > This patch series continues adding arb_gpu_shader_fp64 support to the > Intel driver. Specifically, this targets the i965 scalar backend for > BDW+ hardware (vec4 is still under research and gen7 has its own > issues which we intend tackle after gen

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-02 Thread Jordan Justen
On 2016-05-01 22:47:40, Jordan Justen wrote: > 7-10, 12-20, 36-43, 57-58: > Reviewed-by: Jordan Justen 34-35 Reviewed-by: Jordan Justen > > I also sent questions about 56 & 59. > > On 2016-04-29 04:28:57, Samuel Iglesias Gonsálvez wrote: > > Hello, > > > > This patch series continues adding

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-02 Thread Mark Janes
I neglected to include a cpu test in my list of regressions: piglit.spec.glsl-1_10.compiler.vector-dereference-in-dereference.frag asserts with "glslparsertest: src/mesa/drivers/dri/i965/brw_fs_channel_expressions.cpp:422: virtual ir_visitor_status ir_channel_expressions_visitor::visit_le

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-02 Thread Mark Janes
Samuel Iglesias Gonsálvez writes: > Hello, > > This patch series continues adding arb_gpu_shader_fp64 support to the > Intel driver. Specifically, this targets the i965 scalar backend for > BDW+ hardware (vec4 is still under research and gen7 has its own > issues which we intend tackle after gen

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-05-01 Thread Jordan Justen
7-10, 12-20, 36-43, 57-58: Reviewed-by: Jordan Justen I also sent questions about 56 & 59. On 2016-04-29 04:28:57, Samuel Iglesias Gonsálvez wrote: > Hello, > > This patch series continues adding arb_gpu_shader_fp64 support to the > Intel driver. Specifically, this targets the i965 scalar back

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-04-30 Thread Samuel Iglesias Gonsálvez
On 2016-04-30 10:34, Kenneth Graunke wrote: With my horiz_offset concerns and other minor comments addressed, Patches 1-33 (except 2, 3, 11, and 22) and 36, 40-55, 57-58 are: Reviewed-by: Kenneth Graunke I'll look at the rest soon. It sounds like Curro is taking a look too, so he may have mor

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-04-30 Thread Kenneth Graunke
With my horiz_offset concerns and other minor comments addressed, Patches 1-33 (except 2, 3, 11, and 22) and 36, 40-55, 57-58 are: Reviewed-by: Kenneth Graunke I'll look at the rest soon. It sounds like Curro is taking a look too, so he may have more feedback. signature.asc Description: This

Re: [Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-04-30 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > Hello, > > This patch series continues adding arb_gpu_shader_fp64 support to the > Intel driver. Specifically, this targets the i965 scalar backend for > BDW+ hardware (vec4 is still under research and gen7 has its own > issues which we intend tackle after gen

[Mesa-dev] [PATCH 00/59] Initial arb_gpu_shader_fp64 support to the i965 scalar backend

2016-04-29 Thread Samuel Iglesias Gonsálvez
Hello, This patch series continues adding arb_gpu_shader_fp64 support to the Intel driver. Specifically, this targets the i965 scalar backend for BDW+ hardware (vec4 is still under research and gen7 has its own issues which we intend tackle after gen8). This adds most of the fp64 scalar implemen