Re: [igt-dev] [PATCH i-g-t 4/5] autoconf: Silence void pointer arithmetic warnings

From: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>, igt-dev@lists.freedesktop.org
Cc: Intel-gfx@lists.freedesktop.org,
	Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Subject: Re: [igt-dev] [PATCH i-g-t 4/5] autoconf: Silence void pointer arithmetic warnings
Date: Thu, 28 Feb 2019 14:42:09 +0000	[thread overview]
Message-ID: <9b6cd18d-1e9e-5b99-b293-8a90dd2e1adb@linux.intel.com> (raw)
In-Reply-To: <155136429518.5847.12808789372500594905@skylake-alporthouse-com>

On 28/02/2019 14:31, Chris Wilson wrote:
> Quoting Tvrtko Ursulin (2019-02-28 14:18:27)
>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>
>> Meson build does it so make the two symmetrical in this respect.
>>
>> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>> ---
>>   configure.ac | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 4f55ea5d0f89..210e2c57df55 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -256,6 +256,8 @@ AC_ARG_ENABLE(werror,
>>                               [Fail on warnings]),
>>                [], [enable_werror=no])
>>   
>> +CWARNFLAGS="$CWARNFLAGS -Wno-pointer-arith"
> 
> \o/

Maybe I am only testing to see who still uses automake. :))

>> +
>>   if test "x$enable_debug" = xyes; then
>>          AS_COMPILER_FLAG([-g3], [DEBUG_CFLAGS="-g3"], [DEBUG_CFLAGS="-g"])
>>          AS_COMPILER_FLAG([-Og], [DEBUG_CFLAGS+=" -Og -Wno-maybe-uninitialized"], # disable maybe-uninitialized due to false positives
>> -- 
> 
> But shouldn't we be using something like the above to verify the flag
> exists?

Definitely true.

Regards,

Tvrtko
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev