On Sun, Oct 19, 2014 at 11:41 AM, Emil Velikov <emil.l.veli...@gmail.com> 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 explanatory
> external/mesa/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp:421:35:
> *error: 'isfinite' was not declared in this scope*
>
> Whereas which branch of the ifdef spaghetti the compiler takes in order
> to get there is a good question. I doubt that I'll get the chance to
Well, u_math.h actually only provides an isfinite for msvc (and
assumes the rest will have it).
> look into it any time soon. Yet it makes sense to use C++ headers for
> C++ code - don't you agree ?
The current contract is that u_math.h makes isfinite work. There's
other code in the tree that uses isfinite (admittedly only in vega).
I'd be perfectly happy with this patch if it also removed isfinite
references from u_math.h. However as long as u_math.h provides some
isfinite support, it should be fixed there.
-ilia
>> On Sun, Oct 19, 2014 at 11:16 AM, Emil Velikov <emil.l.veli...@gmail.com>
>> wrote:
>>> From: Mauro Rossi <issor.or...@gmail.com>
>>>
>>> Otherwise the android build will fail.
>>>
>>> Cc: "10.2 10.3" <mesa-sta...@lists.freedesktop.org>
>>> ---
>>> 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/nouveau/codegen/nv50_ir_peephole.cpp
>>> index 719f980..b9744e3 100644
>>> --- a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
>>> +++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
>>> @@ -24,9 +24,7 @@
>>> #include "codegen/nv50_ir_target.h"
>>> #include "codegen/nv50_ir_build_util.h"
>>>
>>> -extern "C" {
>>> -#include "util/u_math.h"
>>> -}
>>> +#include <cmath>
>>>
>>> namespace nv50_ir {
>>>
>>> @@ -416,9 +414,9 @@ ConstantFolding::expr(Instruction *i,
>>> case OP_FMA:
>>> case OP_MUL:
>>> if (i->dnz && i->dType == TYPE_F32) {
>>> - if (!isfinite(a->data.f32))
>>> + if (!std::isfinite(a->data.f32))
>>> a->data.f32 = 0.0f;
>>> - if (!isfinite(b->data.f32))
>>> + if (!std::isfinite(b->data.f32))
>>> b->data.f32 = 0.0f;
>>> }
>>> switch (i->dType) {
>>> --
>>> 2.1.0
>>>
>>> _______________________________________________
>>> mesa-dev mailing list
>>> mesa-dev@lists.freedesktop.org
>>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
