Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources

2014-10-23 Thread Emil Velikov
; From: mesa-dev on behalf of Emil > Velikov > Sent: 19 October 2014 08:41 > To: Ilia Mirkin > Cc: mesa-dev@lists.freedesktop.org; Mauro Rossi; emil.l.veli...@gmail.com; > 10.2 10.3 > Subject: Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ > sourc

Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources

2014-10-20 Thread Jose Fonseca
ctober 2014 08:41 To: Ilia Mirkin Cc: mesa-dev@lists.freedesktop.org; Mauro Rossi; emil.l.veli...@gmail.com; 10.2 10.3 Subject: Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources On 19/10/14 16:23, Ilia Mirkin wrote: > Can you provide more detail as to why the u

Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources

2014-10-19 Thread Ilia Mirkin
On Sun, Oct 19, 2014 at 1:23 PM, Emil Velikov wrote: > On 19/10/14 16:59, Ilia Mirkin wrote: >> On Sun, Oct 19, 2014 at 11:41 AM, Emil Velikov >> wrote: >>> On 19/10/14 16:23, Ilia Mirkin wrote: Can you provide more detail as to why the util/u_math.h include is insufficient? Perhaps it

Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources

2014-10-19 Thread Emil Velikov
On 19/10/14 16:59, Ilia Mirkin wrote: > On Sun, Oct 19, 2014 at 11:41 AM, Emil Velikov > wrote: >> On 19/10/14 16:23, Ilia Mirkin wrote: >>> Can you provide more detail as to why the util/u_math.h include is >>> insufficient? Perhaps it should be fixed up similarly to how >>> mesa/main/imports.h

Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources

2014-10-19 Thread Ilia Mirkin
On Sun, Oct 19, 2014 at 11:41 AM, Emil Velikov wrote: > On 19/10/14 16:23, Ilia Mirkin wrote: >> Can you provide more detail as to why the util/u_math.h include is >> insufficient? Perhaps it should be fixed up similarly to how >> mesa/main/imports.h does it? >> > The actual error message is quite

Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources

2014-10-19 Thread Emil Velikov
On 19/10/14 16:23, Ilia Mirkin wrote: > Can you provide more detail as to why the util/u_math.h include is > insufficient? Perhaps it should be fixed up similarly to how > mesa/main/imports.h does it? > The actual error message is quite explanatory external/mesa/src/gallium/drivers/nouveau/codegen

Re: [Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources

2014-10-19 Thread Ilia Mirkin
Can you provide more detail as to why the util/u_math.h include is insufficient? Perhaps it should be fixed up similarly to how mesa/main/imports.h does it? -ilia On Sun, Oct 19, 2014 at 11:16 AM, Emil Velikov wrote: > From: Mauro Rossi > > Otherwise the android build will fail. > > Cc: "10.2

[Mesa-dev] [PATCH 2/3] gallium/nouveau: use std::isfinite in c++ sources

2014-10-19 Thread Emil Velikov
From: Mauro Rossi Otherwise the android build will fail. Cc: "10.2 10.3" --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src/gallium/drivers/nouv