[Mesa-dev] [PATCH] nv50/codegen/nv50_ir.cpp: #include <typeinfo> on -DNDEBUG.

[Christoph Bumiller]

On 31.12.2012 14:01, Christoph Bumiller wrote:
> On 31.12.2012 04:21, Johannes Obermayr wrote:
>> Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=58879
>> ---
>> On debug builds it is included:
>> nv50_ir.cpp:23 -> nv50_ir.h:33 -> nv50_ir_util.h:33 -> <typeinfo>
> And on non-debug builds it's supposed to *not* be included, because they
> should work without rtti.
> Since when are assertions not removed when NDEBUG is defined ?
>
Ok I get it, it's implemented as do { } while(0 && condition) ... Let's
just wrap that assertion in #ifndef NDEBUG then.


>> ---
>>  src/gallium/drivers/nv50/codegen/nv50_ir.cpp |    4 ++++
>>  1 Datei geändert, 4 Zeilen hinzugefügt(+)
>>
>> diff --git a/src/gallium/drivers/nv50/codegen/nv50_ir.cpp b/src/gallium/drivers/nv50/codegen/nv50_ir.cpp
>> index 335e9e0..0bce2ce 100644
>> --- a/src/gallium/drivers/nv50/codegen/nv50_ir.cpp
>> +++ b/src/gallium/drivers/nv50/codegen/nv50_ir.cpp
>> @@ -24,6 +24,10 @@
>>  #include "nv50_ir_target.h"
>>  #include "nv50_ir_driver.h"
>>  
>> +#ifdef NDEBUG
>> +# include <typeinfo>
>> +#endif 
>> +
>>  extern "C" {
>>  #include "nv50/nv50_program.h"
>>  #include "nv50/nv50_debug.h"
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>