Bug 1481719 - Disable profile-instr-out-of-date warnings explicitly rather than allowing all warnings where they appear. r=dmajor

This commit is contained in:
Mike Hommey 2018-08-08 14:49:53 +09:00
parent fe9dd50d7f
commit 6d197a0707
9 changed files with 2 additions and 23 deletions

View file

@ -98,6 +98,3 @@ if CONFIG['CC_TYPE'] == 'clang-cl':
'-Wno-missing-braces', '-Wno-missing-braces',
'-Wno-unused-const-variable', '-Wno-unused-const-variable',
] ]
if CONFIG['MOZ_PGO'] and CONFIG['CC_TYPE'] == 'clang-cl':
AllowCompilerWarnings() # workaround for bug 1090497

View file

@ -44,6 +44,3 @@ if CONFIG['CC_TYPE'] == 'clang-cl':
'-Wno-missing-braces', '-Wno-missing-braces',
'-Wno-unused-const-variable', '-Wno-unused-const-variable',
] ]
if CONFIG['MOZ_PGO'] and CONFIG['CC_TYPE'] == 'clang-cl':
AllowCompilerWarnings() # workaround for bug 1090497

View file

@ -1267,6 +1267,7 @@ def pgo_flags(compiler, build_env, target):
gen_cflags=['-fprofile-instr-generate'], gen_cflags=['-fprofile-instr-generate'],
gen_ldflags=['clang_rt.profile-x86_64.lib'], gen_ldflags=['clang_rt.profile-x86_64.lib'],
use_cflags=['-fprofile-instr-use=%s' % profdata, use_cflags=['-fprofile-instr-use=%s' % profdata,
'-Wno-error=profile-instr-out-of-date',
'-Wno-error=profile-instr-unprofiled'], '-Wno-error=profile-instr-unprofiled'],
use_ldflags=[], use_ldflags=[],
) )

View file

@ -26,7 +26,6 @@ OS_LIBS += [
'comctl32', 'comctl32',
] ]
if ((CONFIG['MOZ_PGO'] or if (not CONFIG['HAVE_64BIT_BUILD'] and CONFIG['ENABLE_CLANG_PLUGIN']
(not CONFIG['HAVE_64BIT_BUILD'] and CONFIG['ENABLE_CLANG_PLUGIN']))
and CONFIG['CC_TYPE'] == 'clang-cl'): and CONFIG['CC_TYPE'] == 'clang-cl'):
AllowCompilerWarnings() # workaround for bug 1090497 AllowCompilerWarnings() # workaround for bug 1090497

View file

@ -30,6 +30,3 @@ BROWSER_CHROME_MANIFESTS += [ 'tests/browser.ini' ]
include('/ipc/chromium/chromium-config.mozbuild') include('/ipc/chromium/chromium-config.mozbuild')
FINAL_LIBRARY = 'xul' FINAL_LIBRARY = 'xul'
if CONFIG['MOZ_PGO'] and CONFIG['CC_TYPE'] == 'clang-cl':
AllowCompilerWarnings() # workaround for bug 1090497

View file

@ -23,6 +23,3 @@ if CONFIG['MOZ_SANDBOX'] and CONFIG['OS_ARCH'] == 'WINNT':
'/security/sandbox/chromium', '/security/sandbox/chromium',
'/security/sandbox/chromium-shim', '/security/sandbox/chromium-shim',
] ]
if CONFIG['MOZ_PGO'] and CONFIG['CC_TYPE'] == 'clang-cl':
AllowCompilerWarnings() # workaround for bug 1090497

View file

@ -53,6 +53,3 @@ if CONFIG['OS_ARCH'] == 'WINNT':
CFLAGS += CONFIG['TK_CFLAGS'] CFLAGS += CONFIG['TK_CFLAGS']
CXXFLAGS += CONFIG['TK_CFLAGS'] CXXFLAGS += CONFIG['TK_CFLAGS']
OS_LIBS += CONFIG['TK_LIBS'] OS_LIBS += CONFIG['TK_LIBS']
if CONFIG['MOZ_PGO'] and CONFIG['CC_TYPE'] == 'clang-cl':
AllowCompilerWarnings() # workaround for bug 1090497

View file

@ -13,6 +13,3 @@ UNIFIED_SOURCES = [
if CONFIG['COMPILE_ENVIRONMENT']: if CONFIG['COMPILE_ENVIRONMENT']:
TEST_HARNESS_FILES.gtest += ['!InjectorDLL.dll'] TEST_HARNESS_FILES.gtest += ['!InjectorDLL.dll']
if CONFIG['MOZ_PGO'] and CONFIG['CC_TYPE'] == 'clang-cl':
AllowCompilerWarnings() # workaround for bug 1090497

View file

@ -59,6 +59,3 @@ if CONFIG['OS_TARGET'] != 'WINNT':
DisableStlWrapping() DisableStlWrapping()
include('/toolkit/crashreporter/crashreporter.mozbuild') include('/toolkit/crashreporter/crashreporter.mozbuild')
if CONFIG['MOZ_PGO'] and CONFIG['CC_TYPE'] == 'clang-cl':
AllowCompilerWarnings() # workaround for bug 1090497