← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to m4/pandora_warnings.m4

Merge Joe, plus I updated the tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
m4/pandora_warnings.m4
1
 
dnl  Copyright (C) 2009 Sun Microsystems, Inc.
2
 
dnl This file is free software; Sun Microsystems, Inc.
 
1
dnl  Copyright (C) 2009 Sun Microsystems
 
2
dnl This file is free software; Sun Microsystems
3
3
dnl gives unlimited permission to copy and/or distribute it,
4
4
dnl with or without modifications, as long as this notice is preserved.
5
5
 
34
34
          [ac_cv_warnings_as_errors=yes],
35
35
          [ac_cv_warnings_as_errors=no]))
36
36
 
37
 
  AC_ARG_ENABLE([gcc-profile-mode],
38
 
      [AS_HELP_STRING([--enable-gcc-profile-mode],
39
 
         [Toggle gcc profile mode @<:@default=off@:>@])],
40
 
      [ac_gcc_profile_mode="$enableval"],
41
 
      [ac_gcc_profile_mode="no"])
42
 
 
43
37
  AC_ARG_ENABLE([profiling],
44
38
      [AS_HELP_STRING([--enable-profiling],
45
39
         [Toggle profiling @<:@default=off@:>@])],
93
87
            F_DIAGNOSTICS_SHOW_OPTION="-fdiagnostics-show-option"
94
88
          ])
95
89
 
96
 
    AC_CACHE_CHECK([whether it is safe to use -floop-parallelize-all],
97
 
      [ac_cv_safe_to_use_floop_parallelize_all_],
 
90
    AC_CACHE_CHECK([whether it is safe to use -Wconversion],
 
91
      [ac_cv_safe_to_use_wconversion_],
98
92
      [save_CFLAGS="$CFLAGS"
99
 
       CFLAGS="-floop-parallelize-all ${AM_CFLAGS} ${CFLAGS}"
 
93
       dnl Use -Werror here instead of ${W_FAIL} so that we don't spew
 
94
       dnl conversion warnings to all the tarball folks
 
95
       CFLAGS="-Wconversion -Werror -pedantic ${AM_CFLAGS} ${CFLAGS}"
100
96
       AC_COMPILE_IFELSE(
101
 
         [AC_LANG_PROGRAM([],[])],
102
 
         [ac_cv_safe_to_use_floop_parallelize_all_=yes],
103
 
         [ac_cv_safe_to_use_floop_parallelize_all_=no])
 
97
         [AC_LANG_PROGRAM([[
 
98
#include <stdbool.h>
 
99
void foo(bool a)
 
100
{
 
101
  (void)a;
 
102
}
 
103
         ]],[[
 
104
foo(0);
 
105
         ]])],
 
106
         [ac_cv_safe_to_use_wconversion_=yes],
 
107
         [ac_cv_safe_to_use_wconversion_=no])
104
108
       CFLAGS="$save_CFLAGS"])
105
109
 
106
 
    AS_IF([test "$ac_cv_safe_to_use_floop_parallelize_all_" = "yes"],
107
 
          [
108
 
            F_LOOP_PARALLELIZE_ALL="-floop-parallelize-all"
109
 
          ])
 
110
    AS_IF([test "$ac_cv_safe_to_use_wconversion_" = "yes"],
 
111
      [W_CONVERSION="-Wconversion"
 
112
      AC_CACHE_CHECK([whether it is safe to use -Wconversion with htons],
 
113
        [ac_cv_safe_to_use_Wconversion_],
 
114
        [save_CFLAGS="$CFLAGS"
 
115
         dnl Use -Werror here instead of ${W_FAIL} so that we don't spew
 
116
         dnl conversion warnings to all the tarball folks
 
117
         CFLAGS="-Wconversion -Werror -pedantic ${AM_CFLAGS} ${CFLAGS}"
 
118
         AC_COMPILE_IFELSE(
 
119
           [AC_LANG_PROGRAM(
 
120
             [[
 
121
#include <netinet/in.h>
 
122
             ]],[[
 
123
uint16_t x= htons(80);
 
124
             ]])],
 
125
           [ac_cv_safe_to_use_Wconversion_=yes],
 
126
           [ac_cv_safe_to_use_Wconversion_=no])
 
127
         CFLAGS="$save_CFLAGS"])
 
128
 
 
129
      AS_IF([test "$ac_cv_safe_to_use_Wconversion_" = "no"],
 
130
            [NO_CONVERSION="-Wno-conversion"])
 
131
    ])
110
132
 
111
133
    NO_STRICT_ALIASING="-fno-strict-aliasing -Wno-strict-aliasing"
112
134
    NO_SHADOW="-Wno-shadow"
113
135
 
114
136
    AS_IF([test "$INTELCC" = "yes"],[
115
137
      m4_if(PW_LESS_WARNINGS,[no],[
116
 
        BASE_WARNINGS="-w1 -Werror -Wcheck -Wp64 -Woverloaded-virtual -Wcast-qual -diag-disable 188"
 
138
        BASE_WARNINGS="-w1 -Werror -Wcheck -Wformat -Wp64 -Woverloaded-virtual -Wcast-qual"
117
139
      ],[
118
 
        dnl 2203 is like old-style-cast
119
 
        dnl 1684 is like strict-aliasing
120
 
        dnl 188 is about using enums as bitfields
121
 
        dnl 1683 is a warning about _EXPLICIT_ casting, which we want
122
 
        BASE_WARNINGS="-w1 -Werror -Wcheck -Wp64 -Woverloaded-virtual -Wcast-qual -diag-disable 188,981,2259,2203,1683,1684"
 
140
        BASE_WARNINGS="-w1 -Wcheck -Wformat -Wp64 -Woverloaded-virtual -Wcast-qual -diag-disable 981"
123
141
      ])
124
142
      CC_WARNINGS="${BASE_WARNINGS}"
125
143
      CXX_WARNINGS="${BASE_WARNINGS}"
126
 
      PROTOSKIP_WARNINGS="-diag-disable 188,981,967,2259,1683,1684,2203"
127
 
      
128
144
    ],[
129
145
      m4_if(PW_LESS_WARNINGS,[no],[
130
 
        BASE_WARNINGS_FULL="${W_CONVERSION} -Wstrict-aliasing"
 
146
        BASE_WARNINGS_FULL="-Wformat=2 ${W_CONVERSION} -Wstrict-aliasing"
131
147
        CC_WARNINGS_FULL="-Wswitch-default -Wswitch-enum -Wwrite-strings"
132
148
        CXX_WARNINGS_FULL="-Weffc++ -Wold-style-cast"
133
149
        NO_OLD_STYLE_CAST="-Wno-old-style-cast"
134
150
        NO_EFF_CXX="-Wno-effc++"
135
151
      ],[
136
 
        BASE_WARNINGS_FULL="${NO_STRICT_ALIASING}"
 
152
        BASE_WARNINGS_FULL="-Wformat ${NO_STRICT_ALIASING}"
137
153
      ])
138
154
 
139
155
      AS_IF([test "${ac_cv_assert}" = "no"],
153
169
        [ac_cv_safe_to_use_Wextra_=no])
154
170
      CFLAGS="$save_CFLAGS"])
155
171
 
156
 
      BASE_WARNINGS="${W_FAIL} -pedantic -Wall -Wundef -Wshadow ${NO_UNUSED} ${F_DIAGNOSTICS_SHOW_OPTION} ${F_LOOP_PARALLELIZE_ALL} ${BASE_WARNINGS_FULL}"
 
172
      BASE_WARNINGS="${W_FAIL} -pedantic -Wall -Wundef -Wshadow ${NO_UNUSED} ${F_DIAGNOSTICS_SHOW_OPTION} ${CFLAG_VISIBILITY} ${BASE_WARNINGS_FULL}"
157
173
      AS_IF([test "$ac_cv_safe_to_use_Wextra_" = "yes"],
158
174
            [BASE_WARNINGS="${BASE_WARNINGS} -Wextra"],
159
175
            [BASE_WARNINGS="${BASE_WARNINGS} -W"])
160
176
  
161
 
      AC_CACHE_CHECK([whether it is safe to use -Wformat],
162
 
        [ac_cv_safe_to_use_wformat_],
163
 
        [save_CFLAGS="$CFLAGS"
164
 
         dnl Use -Werror here instead of ${W_FAIL} so that we don't spew
165
 
         dnl conversion warnings to all the tarball folks
166
 
         CFLAGS="-Wformat -Werror -pedantic ${AM_CFLAGS} ${CFLAGS}"
167
 
         AC_COMPILE_IFELSE(
168
 
           [AC_LANG_PROGRAM([[
169
 
#include <stdio.h>
170
 
#include <stdint.h>
171
 
#include <inttypes.h>
172
 
void foo();
173
 
void foo()
174
 
{
175
 
  uint64_t test_u= 0;
176
 
  printf("This is a %" PRIu64 "test\n", test_u);
177
 
}
178
 
           ]],[[
179
 
foo();
180
 
           ]])],
181
 
           [ac_cv_safe_to_use_wformat_=yes],
182
 
           [ac_cv_safe_to_use_wformat_=no])
183
 
         CFLAGS="$save_CFLAGS"])
184
 
      AS_IF([test "$ac_cv_safe_to_use_wformat_" = "yes"],[
185
 
        BASE_WARNINGS="${BASE_WARNINGS} -Wformat -Wno-format-nonliteral -Wno-format-security"
186
 
        BASE_WARNINGS_FULL="${BASE_WARNINGS_FULL} -Wformat=2 -Wno-format-nonliteral -Wno-format-security"
187
 
        ],[
188
 
        BASE_WARNINGS="${BASE_WARNINGS} -Wno-format"
189
 
        BASE_WARNINGS_FULL="${BASE_WARNINGS_FULL} -Wno-format"
190
 
      ])
191
 
 
192
 
 
193
 
 
194
 
      AC_CACHE_CHECK([whether it is safe to use -Wconversion],
195
 
        [ac_cv_safe_to_use_wconversion_],
196
 
        [save_CFLAGS="$CFLAGS"
197
 
         dnl Use -Werror here instead of ${W_FAIL} so that we don't spew
198
 
         dnl conversion warnings to all the tarball folks
199
 
         CFLAGS="-Wconversion -Werror -pedantic ${AM_CFLAGS} ${CFLAGS}"
200
 
         AC_COMPILE_IFELSE(
201
 
           [AC_LANG_PROGRAM([[
202
 
#include <stdbool.h>
203
 
void foo(bool a)
204
 
{
205
 
  (void)a;
206
 
}
207
 
           ]],[[
208
 
foo(0);
209
 
           ]])],
210
 
           [ac_cv_safe_to_use_wconversion_=yes],
211
 
           [ac_cv_safe_to_use_wconversion_=no])
212
 
         CFLAGS="$save_CFLAGS"])
213
 
  
214
 
      AS_IF([test "$ac_cv_safe_to_use_wconversion_" = "yes"],
215
 
        [W_CONVERSION="-Wconversion"
216
 
        AC_CACHE_CHECK([whether it is safe to use -Wconversion with htons],
217
 
          [ac_cv_safe_to_use_Wconversion_],
218
 
          [save_CFLAGS="$CFLAGS"
219
 
           dnl Use -Werror here instead of ${W_FAIL} so that we don't spew
220
 
           dnl conversion warnings to all the tarball folks
221
 
           CFLAGS="-Wconversion -Werror -pedantic ${AM_CFLAGS} ${CFLAGS}"
222
 
           AC_COMPILE_IFELSE(
223
 
             [AC_LANG_PROGRAM(
224
 
               [[
225
 
#include <netinet/in.h>
226
 
               ]],[[
227
 
uint16_t x= htons(80);
228
 
               ]])],
229
 
             [ac_cv_safe_to_use_Wconversion_=yes],
230
 
             [ac_cv_safe_to_use_Wconversion_=no])
231
 
           CFLAGS="$save_CFLAGS"])
232
 
  
233
 
        AS_IF([test "$ac_cv_safe_to_use_Wconversion_" = "no"],
234
 
              [NO_CONVERSION="-Wno-conversion"])
235
 
      ])
236
 
 
237
177
      CC_WARNINGS="${BASE_WARNINGS} -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wmissing-declarations -Wcast-align ${CC_WARNINGS_FULL}"
238
178
      CXX_WARNINGS="${BASE_WARNINGS} -Woverloaded-virtual -Wnon-virtual-dtor -Wctor-dtor-privacy -Wno-long-long ${CXX_WARNINGS_FULL}"
239
179
 
256
196
      AS_IF([test "$ac_cv_safe_to_use_Wmissing_declarations_" = "yes"],
257
197
            [CXX_WARNINGS="${CXX_WARNINGS} -Wmissing-declarations"])
258
198
  
259
 
      AC_CACHE_CHECK([whether it is safe to use -Wframe-larger-than],
260
 
        [ac_cv_safe_to_use_Wframe_larger_than_],
261
 
        [AC_LANG_PUSH(C++)
262
 
         save_CXXFLAGS="$CXXFLAGS"
263
 
         CXXFLAGS="-Werror -pedantic -Wframe-larger-than=32768 ${AM_CXXFLAGS}"
264
 
         AC_COMPILE_IFELSE([
265
 
           AC_LANG_PROGRAM(
266
 
           [[
267
 
#include <stdio.h>
268
 
           ]], [[]])
269
 
        ],
270
 
        [ac_cv_safe_to_use_Wframe_larger_than_=yes],
271
 
        [ac_cv_safe_to_use_Wframe_larger_than_=no])
272
 
        CXXFLAGS="$save_CXXFLAGS"
273
 
        AC_LANG_POP()
274
 
      ])
275
 
      AS_IF([test "$ac_cv_safe_to_use_Wframe_larger_than_" = "yes"],
276
 
            [CXX_WARNINGS="${CXX_WARNINGS} -Wframe-larger-than=32768"])
277
 
  
278
199
      AC_CACHE_CHECK([whether it is safe to use -Wlogical-op],
279
200
        [ac_cv_safe_to_use_Wlogical_op_],
280
201
        [save_CFLAGS="$CFLAGS"
371
292
      dnl TODO: Figure out a better way to deal with this:
372
293
      PROTOSKIP_WARNINGS="-Wno-effc++ -Wno-shadow -Wno-missing-braces ${NO_ATTRIBUTES}"
373
294
      NO_WERROR="-Wno-error"
374
 
      PERMISSIVE_WARNINGS="-Wno-error -Wno-unused-function -fpermissive"
375
 
      PERMISSIVE_C_WARNINGS="-Wno-error -Wno-redundant-decls"
 
295
      INNOBASE_SKIP_WARNINGS="-Wno-shadow -Wno-cast-align"
376
296
      AS_IF([test "$host_vendor" = "apple"],[
377
297
        BOOSTSKIP_WARNINGS="-Wno-uninitialized"
378
298
      ])
406
326
 
407
327
 
408
328
    m4_if(PW_LESS_WARNINGS, [no],[
409
 
      CC_WARNINGS_FULL="-erroff=E_STATEMENT_NOT_REACHED,E_INTEGER_OVERFLOW_DETECTED${W_PASTE_RESULT}"
 
329
      CC_WARNINGS_FULL="-erroff=E_INTEGER_OVERFLOW_DETECTED${W_PASTE_RESULT}"
410
330
      CXX_WARNINGS_FULL="-erroff=inllargeuse"
411
331
    ],[
412
 
      CC_WARNINGS_FULL="-erroff=E_ATTRIBUTE_NOT_VAR,E_STATEMENT_NOT_REACHED"
 
332
      CC_WARNINGS_FULL="-erroff=E_ATTRIBUTE_NOT_VAR"
413
333
      CXX_WARNINGS_FULL="-erroff=attrskipunsup,doubunder,reftotemp,inllargeuse,truncwarn1,signextwarn,inllargeint"
414
334
    ])
415
335
 
416
 
    CC_WARNINGS="-v -errtags=yes ${W_FAIL} ${CC_WARNINGS_FULL}"
417
 
    CXX_WARNINGS="+w +w2 -xwe -xport64 -errtags=yes ${CXX_WARNINGS_FULL} ${W_FAIL}"
 
336
    CC_WARNINGS="-v -errtags=yes ${W_FAIL} ${CC_WARNINGS_FULL} ${CFLAG_VISIBILITY}"
 
337
    CXX_WARNINGS="+w +w2 -xwe -xport64 -errtags=yes ${CXX_WARNINGS_FULL} ${W_FAIL} ${CFLAG_VISIBILITY}"
418
338
    PROTOSKIP_WARNINGS="-erroff=attrskipunsup,doubunder,reftotemp,wbadinitl,identexpected,inllargeuse,truncwarn1,signextwarn,partinit,notused,badargtype2w,wbadinit"
419
339
    BOOSTSKIP_WARNINGS="-erroff=attrskipunsup,doubunder,reftotemp,inllargeuse,truncwarn1,signextwarn,inllargeint,hidef,wvarhidenmem"
420
 
    PERMISSIVE_WARNINGS="-erroff=attrskipunsup,doubunder,reftotemp,inllargeuse,truncwarn1,signextwarn,inllargeint,hidef,wvarhidenmem,notused,badargtype2w,wunreachable"
421
 
    INNOBASE_SKIP_WARNINGS="-erroff=attrskipunsup,doubunder,reftotemp,wbadinitl,identexpected,inllargeuse,truncwarn1,signextwarn,partinit,notused,badargtype2w,wbadinit,wunreachable"
422
340
    NO_UNREACHED="-erroff=E_STATEMENT_NOT_REACHED"
423
341
    NO_WERROR="-errwarn=%none"
424
342
 
432
350
  AC_SUBST(NO_EFF_CXX)
433
351
  AC_SUBST(NO_OLD_STYLE_CAST)
434
352
  AC_SUBST(PROTOSKIP_WARNINGS)
 
353
  AC_SUBST(BOOSTSKIP_WARNINGS)
435
354
  AC_SUBST(INNOBASE_SKIP_WARNINGS)
436
 
  AC_SUBST(BOOSTSKIP_WARNINGS)
437
 
  AC_SUBST(PERMISSIVE_WARNINGS)
438
 
  AC_SUBST(PERMISSIVE_C_WARNINGS)
439
355
  AC_SUBST(NO_WERROR)
440
356
  AC_SUBST([GCOV_LIBS])
441
357