Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-04-13 Thread Jose Fonseca
On 13/04/16 11:21, Ilia Mirkin wrote: Gah, no :( Feel free to revert it. I lost a lot of time on this already. I pushed this out of courtesy to nouveau to avoid build failures until we figure out how to get a generic version to work reliably. He even sent a patch which did this

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-04-13 Thread Ilia Mirkin
Gah, no :( He even sent a patch which did this generally, what was wrong with that one? On Apr 13, 2016 3:07 AM, "Jose Fonseca" wrote: > Pierre, I'm going to commit this one for now, as it seems the safest thing > to do. > > I still think we should try to fix the generic

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-04-13 Thread Jose Fonseca
Pierre, I'm going to commit this one for now, as it seems the safest thing to do. I still think we should try to fix the generic case, but it seems much harder to get this right. I still don't understand how come isinf sometimes appears in the global namespace some times, but not others.

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-25 Thread Pierre Moreau
I'll resend a patch following Jose's suggestion, probably on Sunday. There are just too many things happening before that. Pierre > On 25 Mar 2016, at 00:36, Jose Fonseca wrote: > >> On 24/03/16 22:07, Ilia Mirkin wrote: >>> On Sat, Mar 19, 2016 at 6:30 PM, Jose Fonseca

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-24 Thread Jose Fonseca
On 24/03/16 22:07, Ilia Mirkin wrote: On Sat, Mar 19, 2016 at 6:30 PM, Jose Fonseca wrote: On 19/03/16 22:25, Ilia Mirkin wrote: On Sat, Mar 19, 2016 at 6:23 PM, Jose Fonseca wrote: On 18/03/16 04:00, Ilia Mirkin wrote: On Mar 17, 2016 8:27

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-24 Thread Ilia Mirkin
On Sat, Mar 19, 2016 at 6:30 PM, Jose Fonseca wrote: > On 19/03/16 22:25, Ilia Mirkin wrote: >> >> On Sat, Mar 19, 2016 at 6:23 PM, Jose Fonseca wrote: >>> >>> On 18/03/16 04:00, Ilia Mirkin wrote: On Mar 17, 2016 8:27 PM, "Matt

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-24 Thread Vinson Lee
On Thu, Mar 17, 2016 at 5:25 PM, Samuel Pitoiset wrote: > > > On 03/18/2016 01:22 AM, Matt Turner wrote: >> >> On Thu, Mar 17, 2016 at 5:17 PM, Pierre Moreau >> wrote: >>> >>> This fixes a compile error while building Nouveau with C++11 enabled

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-19 Thread Jose Fonseca
On 19/03/16 22:30, Jose Fonseca wrote: On 19/03/16 22:25, Ilia Mirkin wrote: On Sat, Mar 19, 2016 at 6:23 PM, Jose Fonseca wrote: On 18/03/16 04:00, Ilia Mirkin wrote: On Mar 17, 2016 8:27 PM, "Matt Turner" > wrote: >

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-19 Thread Ilia Mirkin
On Sat, Mar 19, 2016 at 6:23 PM, Jose Fonseca wrote: > On 18/03/16 04:00, Ilia Mirkin wrote: >> >> >> On Mar 17, 2016 8:27 PM, "Matt Turner" > > wrote: >> > >> > On Thu, Mar 17, 2016 at 5:17 PM, Pierre Moreau

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-19 Thread Jose Fonseca
On 19/03/16 22:25, Ilia Mirkin wrote: On Sat, Mar 19, 2016 at 6:23 PM, Jose Fonseca wrote: On 18/03/16 04:00, Ilia Mirkin wrote: On Mar 17, 2016 8:27 PM, "Matt Turner" > wrote: > > On Thu, Mar 17, 2016 at 5:17 PM,

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-19 Thread Jose Fonseca
On 18/03/16 04:00, Ilia Mirkin wrote: On Mar 17, 2016 8:27 PM, "Matt Turner" > wrote: > > On Thu, Mar 17, 2016 at 5:17 PM, Pierre Moreau > wrote: > > This fixes a compile error while building

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-19 Thread Samuel Pitoiset
On 03/18/2016 01:22 AM, Matt Turner wrote: On Thu, Mar 17, 2016 at 5:17 PM, Pierre Moreau wrote: This fixes a compile error while building Nouveau with C++11 enabled (and glibc >= 2.23). This happens if SWR is enabled, as it forces C++11. That seems bad, right?

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-19 Thread Matt Turner
On Thu, Mar 17, 2016 at 5:17 PM, Pierre Moreau wrote: > This fixes a compile error while building Nouveau with C++11 enabled (and > glibc >= 2.23). This happens if SWR is enabled, as it forces C++11. That seems bad, right? Enabling OpenSWR should affect how any other

Re: [Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-19 Thread Ilia Mirkin
On Mar 17, 2016 8:27 PM, "Matt Turner" wrote: > > On Thu, Mar 17, 2016 at 5:17 PM, Pierre Moreau wrote: > > This fixes a compile error while building Nouveau with C++11 enabled (and > > glibc >= 2.23). This happens if SWR is enabled, as it forces C++11.

[Mesa-dev] [PATCH] nv50/ra: `isinf()` is in namespace `std` since C++11

2016-03-18 Thread Pierre Moreau
This fixes a compile error while building Nouveau with C++11 enabled (and glibc >= 2.23). This happens if SWR is enabled, as it forces C++11. Signed-off-by: Pierre Moreau --- src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp | 4 1 file changed, 4 insertions(+)