Support vtable_verify (new compiler option).

base/allocator/allocator.gyp

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     'jemalloc_dir': '../../third_party/jemalloc/chromium',
     'tcmalloc_dir': '../../third_party/tcmalloc/chromium',
+    'use_vtable_verify%': 0,
   },
   'targets': [
     {
       'target_name': 'allocator',
       'type': 'static_library',
       # Make sure the allocation library is optimized to
       # the hilt in official builds.
       'variables': {
         'optimize': 'max',
       },
@@ skipping 368 matching lines @@
             'ldflags': [
               # Don't let linker rip this symbol out, otherwise the heap&cpu
               # profilers will not initialize properly on startup.
               '-Wl,-uIsHeapProfilerRunning,-uProfilerStart',
               # Do the same for heap leak checker.
               '-Wl,-u_Z21InitialMallocHook_NewPKvj,-u_Z22InitialMallocHook_MMapPKvS0_jiiix,-u_Z22InitialMallocHook_SbrkPKvi',
               '-Wl,-u_Z21InitialMallocHook_NewPKvm,-u_Z22InitialMallocHook_MMapPKvS0_miiil,-u_Z22InitialMallocHook_SbrkPKvl',
               '-Wl,-u_ZN15HeapLeakChecker12IgnoreObjectEPKv,-u_ZN15HeapLeakChecker14UnIgnoreObjectEPKv',
           ]},
         }],
+        [ 'use_vtable_verify==1', {
+          'cflags': [
+            '-Wno-sign-compare',
+            '-Wno-unused-result',
+            '-fvtable-verify=preinit',
+          ],
+          'cflags!': [
+            '-fvtable-verify=std',

[Mark Mentovai, 2012/08/15 20:21:38]

OK, so let’s talk more about this. Where’s your “gcc wrapper script” sit in the
pipeline?

[Mark Mentovai, 2012/08/15 20:38:36]

So then this line, which is supposed to exclude -fvtable-verify=std from cflags,
is entirely inoperative. Your wrapper script is passing -fvtable-verify=std to
the actual compiler driver. -fvtable-verify=std is never present in the
configuration in GYP or in the build environment (make, ninja, or whatever
else).

+          ],
+        }],
         [ 'linux_use_debugallocation==1', {
           'sources!': [
             # debugallocation.cc #includes tcmalloc.cc,
             # so only one of them should be used.
             '<(tcmalloc_dir)/src/tcmalloc.cc',
           ],
           'defines': [
             'TCMALLOC_FOR_DEBUGALLOCATION',
           ],
         }, { # linux_use_debugallocation != 1
@@ skipping 118 matching lines @@
         'dependencies': [
           '../../testing/gtest.gyp:gtest',
           '../base.gyp:base',
           'allocator',
         ],
       },
       ],
     }],
   ],
 }