[Mesa-dev] [RFC] [PATCH 00/10] NIR header (C++ compilation) TLC
Emil Velikov
emil.l.velikov at gmail.com
Wed Oct 7 03:11:27 PDT 2015
Hi all,
A while back we had a case where NIR had C99 initializers in its public
headers. That's not desired as ISO C++ does not allow them.
So I took a quick look about wiring up a compilation test at 'make
check' and noticed that we've been missing some include guards, plus
other trivial tidbits.
Note that nir_intrinsics.h is still 'broken' and perhaps we should
convert it (alongside the C file) to python/mako.
With these in place the only warning that GCC produces on my system
(with -pedantic enabled) are
- use of C++11 long long integer constant [-Wlong-long]
- anonymous variadic macros were introduced in C++11 [-Wvariadic-macros]
- comma at end of enumerator list [-Wpedantic]
- ISO C++ forbids compound-literals [-Wpedantic]
- ISO C++ forbids zero-size array ‘src’ [-Wpedantic]
>From which, only the latter two might be of vague interest.
Please review.
Emil
More information about the mesa-dev
mailing list