<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Dec 12, 2018 at 3:52 PM Rob Clark <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Dec 12, 2018 at 3:45 PM Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> wrote:<br>
><br>
> On Wed, Dec 12, 2018 at 3:37 PM Rob Clark <<a href="mailto:robdclark@gmail.com" target="_blank">robdclark@gmail.com</a>> wrote:<br>
>><br>
>> On Wed, Dec 12, 2018 at 3:13 PM Bas Nieuwenhuizen<br>
>> <<a href="mailto:bas@basnieuwenhuizen.nl" target="_blank">bas@basnieuwenhuizen.nl</a>> wrote:<br>
>> ><br>
>> > On Wed, Dec 12, 2018 at 8:59 PM Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> wrote:<br>
>> > ><br>
>> > > There are 2 issues with meson:<br>
>> > > * -DDEBUG is not present in debugoptimized builds.<br>
>> ><br>
>> > Do people expect -DDEBUG for debugoptimized? I would think that debug<br>
>> > optimized would be an optimized build with debug symbols, but not<br>
>> > expensive checks & asserts, which would match the current<br>
>> > debugoptimized build?<br>
>><br>
>> please, no -DDEBUG for debugoptimized.. I use that when I want debug<br>
>> syms but not (for example) nir_validate and other expensive checks.<br>
><br>
><br>
> If nir_validate is so bad, perhaps it shouldn't be run at all. If you work on NIR and it's not important for you to run nir_validate, perhaps it shouldn't be run at all. It doesn't have anything to do with build systems.<br>
><br>
<br>
I do actually want it enabled when I piglit/deqp..  for which I use<br>
debug builds.  But I don't want it if I'm profiling or valgrinding,<br>
where I use debugoptimized..<br></blockquote><div><br></div>At some point, DEBUG will be replaced by !NDEBUG.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Marek<br></div></div>