Backed out changeset 1ec0f839a905 (bug 1086964) for mass Win7 debug test failures.

This commit is contained in:
Ryan VanderMeulen 2018-03-14 20:35:43 -04:00
parent 62cdcecdd3
commit 0007daf65b
6 changed files with 24 additions and 3 deletions

View File

@ -422,13 +422,11 @@ UNIFIED_SOURCES += [
'ipc/UiCompositorControllerParent.cpp',
'ipc/VideoBridgeChild.cpp',
'ipc/VideoBridgeParent.cpp',
'Layers.cpp',
'LayerScope.cpp',
'LayersHelpers.cpp',
'LayersLogging.cpp',
'LayerSorter.cpp',
'LayersTypes.cpp',
'LayerTreeInvalidation.cpp',
'mlgpu/BufferCache.cpp',
'mlgpu/CanvasLayerMLGPU.cpp',
'mlgpu/ContainerLayerMLGPU.cpp',
@ -482,6 +480,8 @@ UNIFIED_SOURCES += [
SOURCES += [
'basic/BasicImageLayer.cpp',
'ImageContainer.cpp',
'Layers.cpp',
'LayerTreeInvalidation.cpp',
'PersistentBufferProvider.cpp',
'protobuf/LayerScopePacket.pb.cc',
'wr/WebRenderTextureHost.cpp',
@ -490,6 +490,14 @@ SOURCES += [
DEFINES['GOOGLE_PROTOBUF_NO_RTTI'] = True
DEFINES['GOOGLE_PROTOBUF_NO_STATIC_INITIALIZER'] = True
# Workaround compiler bug (Bug 795594)
if CONFIG['CC_TYPE'] in ('msvc', 'clang-cl') and CONFIG['CPU_ARCH'] == 'x86_64':
for src in [
'Layers.cpp',
'LayerTreeInvalidation.cpp',
]:
SOURCES[src].no_pgo = True
if CONFIG['MOZ_WIDGET_TOOLKIT'] == 'cocoa':
SOURCES += [
'basic/MacIOSurfaceTextureHostBasic.cpp',

View File

@ -180,7 +180,6 @@ UNIFIED_SOURCES += [
'builtin/Promise.cpp',
'builtin/Reflect.cpp',
'builtin/ReflectParse.cpp',
'builtin/RegExp.cpp',
'builtin/SIMD.cpp',
'builtin/Stream.cpp',
'builtin/String.cpp',
@ -415,6 +414,8 @@ UNIFIED_SOURCES += [
# builtin/Array.cpp and vm/JSAtom.cpp cannot be built in unified mode because
# xpcshell is broken during packaging when compiled with gcc-4.8.2
# builtin/RegExp.cpp cannot be built in unified mode because it is built
# without PGO
# frontend/Parser.cpp cannot be built in unified mode because of explicit
# template instantiations.
# jsmath.cpp cannot be built in unified mode because it needs to re-#define the
@ -430,6 +431,7 @@ UNIFIED_SOURCES += [
# files unlucky enough to be unified with it.
SOURCES += [
'builtin/Array.cpp',
'builtin/RegExp.cpp',
'frontend/Parser.cpp',
'gc/StoreBuffer.cpp',
'jsmath.cpp',
@ -722,6 +724,10 @@ if CONFIG['MOZ_LINKER']:
DEFINES['MOZ_LINKER'] = True
if CONFIG['CC_TYPE'] in ('msvc', 'clang-cl'):
if CONFIG['CPU_ARCH'] == 'x86':
SOURCES['builtin/RegExp.cpp'].no_pgo = True # Bug 772303
elif CONFIG['CPU_ARCH'] == 'x86_64' and CONFIG['JS_HAS_CTYPES']:
SOURCES['ctypes/CTypes.cpp'].no_pgo = True # Bug 810661
# Prevent floating point errors caused by VC++ optimizations
# XXX We should add this to CXXFLAGS, too?
CFLAGS += ['-fp:precise']

View File

@ -30,5 +30,7 @@ if CONFIG['OS_TARGET'] == 'WINNT':
else:
DEFINES['__Userspace_os_%s' % CONFIG['OS_TARGET']] = 1
NO_PGO = True # Don't PGO
if CONFIG['CC_TYPE'] in ('clang', 'gcc'):
CXXFLAGS += ['-Wno-error=shadow']

View File

@ -86,5 +86,7 @@ if CONFIG['OS_TARGET'] == 'OpenBSD':
if CONFIG['OS_TARGET'] == 'DragonFly':
DEFINES['__DragonFly__'] = False
NO_PGO = True # Don't PGO
if CONFIG['CC_TYPE'] in ('clang', 'gcc'):
CFLAGS += ['-std=gnu99']

View File

@ -332,3 +332,5 @@ LOCAL_INCLUDES += [
'../..',
'/xpcom/reflect/xptinfo',
]
NO_PGO = True

View File

@ -34,6 +34,7 @@ else:
'xptcinvoke_asm_x86_msvc.asm',
'xptcstubs.cpp',
]
SOURCES['xptcinvoke.cpp'].no_pgo = True
SOURCES['xptcinvoke_asm_x86_msvc.asm'].flags += ['-safeseh']
FINAL_LIBRARY = 'xul'