New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 605478 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Android arm64 Builder broken on chromium.perf

Project Member Reported by petrcermak@chromium.org, Apr 21 2016

Issue description

Revision range first seen: 388624 (single revision)
Bot: https://build.chromium.org/p/chromium.perf/builders/Android%20arm64%20Builder?numbuilds=200
Link to failing step log: https://build.chromium.org/p/chromium.perf/builders/Android%20arm64%20Builder/builds/36691/steps/compile/logs/stdio


The compile error was clearly caused by https://codereview.chromium.org/1897863006:


FAILED: /b/build/goma/gomacc ../../third_party/android_tools/ndk/toolchains/aarch64-linux-android-4.9/prebuilt/linux-x86_64/bin/aarch64-linux-android-g++ -MMD -MF obj/third_party/WebKit/Source/core/remaining/FrameView.o.d -DV8_DEPRECATION_WARNINGS -DCLD_VERSION=2 -DENABLE_NOTIFICATIONS -DENABLE_BROWSER_CDMS -DENABLE_PRINTING=1 -DENABLE_BASIC_PRINTING=1 -DENABLE_SPELLCHECK=1 -DUSE_BROWSER_SPELLCHECKER=1 -DUSE_OPENSSL_CERTS=1 -DNO_TCMALLOC -DENABLE_WEBRTC=1 -DDISABLE_NACL -DENABLE_SUPERVISED_USERS=1 -DENABLE_AUTOFILL_DIALOG=1 -DUSE_PROPRIETARY_CODECS -DVIDEO_HOLE=1 -DSAFE_BROWSING_DB_REMOTE -DOFFICIAL_BUILD -DGOOGLE_CHROME_BUILD -DENABLE_MEDIA_ROUTER=1 -DENABLE_WEBVR -D_FILE_OFFSET_BITS=64 -DNO_UNWIND_TABLES -DANDROID -DHAVE_SYS_UIO_H -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D_FORTIFY_SOURCE=2 -D__GNU_SOURCE=1 -DNDEBUG -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 -DBLINK_CORE_IMPLEMENTATION=1 -DDISABLE_FFMPEG_VIDEO_DECODERS -DBLINK_IMPLEMENTATION=1 -DINSIDE_BLINK -DSK_IGNORE_LINEONLY_AA_CONVEX_PATH_OPTS -DSK_SUPPORT_GPU=1 -DSK_BUILD_FOR_ANDROID -DUSE_CHROMIUM_SKIA -DENABLE_LAYOUT_UNIT_IN_INLINE_BOXES=0 -DENABLE_OILPAN=1 -DWTF_USE_CONCATENATED_IMPULSE_RESPONSES=1 -DWTF_USE_LOW_QUALITY_IMAGE_INTERPOLATION=1 -DWTF_USE_WEBAUDIO_OPENMAX_DL_FFT=1 -DU_USING_ICU_NAMESPACE=0 -DU_ENABLE_DYLOAD=0 -DU_NOEXCEPT= -DU_STATIC_IMPLEMENTATION -DCHROME_PNG_WRITE_SUPPORT -DPNG_USER_CONFIG -DCHROME_PNG_READ_PACK_SUPPORT -DLIBXSLT_STATIC -I../.. -Igen -I../../third_party/khronos -I../../gpu -I../../third_party/libwebp -I../../third_party/WebKit/Source -Igen/blink -I../../third_party/openmax_dl -I../../third_party/WebKit/Source -I../../third_party/WebKit -Igen/blink -Igen/third_party/WebKit -I../../skia/config -I../../skia/ext -I../../third_party/skia/include/c -I../../third_party/skia/include/config -I../../third_party/skia/include/core -I../../third_party/skia/include/effects -I../../third_party/skia/include/images -I../../third_party/skia/include/lazy -I../../third_party/skia/include/pathops -I../../third_party/skia/include/pdf -I../../third_party/skia/include/pipe -I../../third_party/skia/include/ports -I../../third_party/skia/include/utils -I../../third_party/skia/include/gpu -I../../third_party/skia/src/gpu -I../../third_party/icu/source/common -I../../third_party/icu/source/i18n -I../../third_party/angle/include -Igen/angle -I../../third_party/iccjpeg -I../../third_party/libpng -I../../third_party/zlib -I../../third_party/libxml/src/include -I../../third_party/libxml/linux/include -I../../third_party/libxslt -I../../third_party/ots/include -I../../third_party/qcms/src -I../../third_party/snappy/src -I../../third_party/snappy/linux -I../../v8/include -fno-strict-aliasing --param=ssp-buffer-size=4 -fstack-protector -fno-unwind-tables -fno-asynchronous-unwind-tables -fPIC -pipe -ffunction-sections -fno-short-enums -finline-limit=64 -Wall -Werror -Wno-unused-local-typedefs -Wno-missing-field-initializers -Wno-unused-parameter -Os -fno-ident -fdata-sections -ffunction-sections -g1 --sysroot=../../third_party/android_tools/ndk/platforms/android-21/arch-arm64 -fvisibility=hidden -DLIBXML_STATIC= -fno-threadsafe-statics -fvisibility-inlines-hidden -std=gnu++11 -Wno-narrowing -fno-rtti -isystem../../third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include -isystem../../third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++abi/libcxxabi/include -isystem../../third_party/android_tools/ndk/sources/android/support/include -fno-exceptions -c ../../third_party/WebKit/Source/core/frame/FrameView.cpp -o obj/third_party/WebKit/Source/core/remaining/FrameView.o
In file included from ../../third_party/WebKit/Source/platform/ScriptForbiddenScope.h:11:0,
                 from ../../third_party/WebKit/Source/bindings/core/v8/DOMWrapperMap.h:35,
                 from ../../third_party/WebKit/Source/bindings/core/v8/DOMDataStore.h:34,
                 from ../../third_party/WebKit/Source/bindings/core/v8/V8Binding.h:35,
                 from ../../third_party/WebKit/Source/bindings/core/v8/Dictionary.h:34,
                 from gen/blink/bindings/core/v8/UnionTypesCore.h:10,
                 from ../../third_party/WebKit/Source/core/events/EventTarget.h:36,
                 from ../../third_party/WebKit/Source/core/dom/Node.h:34,
                 from ../../third_party/WebKit/Source/core/dom/ContainerNode.h:29,
                 from ../../third_party/WebKit/Source/core/editing/Position.h:30,
                 from ../../third_party/WebKit/Source/core/editing/PositionWithAffinity.h:9,
                 from ../../third_party/WebKit/Source/core/layout/HitTestResult.h:26,
                 from ../../third_party/WebKit/Source/core/layout/HitTestCache.h:9,
                 from ../../third_party/WebKit/Source/core/layout/LayoutView.h:26,
                 from ../../third_party/WebKit/Source/core/layout/api/LayoutViewItem.h:9,
                 from ../../third_party/WebKit/Source/core/frame/FrameView.h:35,
                 from ../../third_party/WebKit/Source/core/frame/FrameView.cpp:27:
../../third_party/WebKit/Source/wtf/TemporaryChange.h: In member function 'void blink::FrameView::updateLifecyclePhasesInternal(blink::FrameView::LifeCycleUpdateOption)':
../../third_party/WebKit/Source/wtf/TemporaryChange.h:55:9: error: '*((void*)& isUpdatingAllLifecyclePhasesScope +16)' may be used uninitialized in this function [-Werror=maybe-uninitialized]
         m_scopedVariable = m_originalValue;
         ^
../../third_party/WebKit/Source/core/frame/FrameView.cpp:2404:37: note: '*((void*)& isUpdatingAllLifecyclePhasesScope +16)' was declared here
     Optional<TemporaryChange<bool>> isUpdatingAllLifecyclePhasesScope;
                                     ^
In file included from ../../third_party/WebKit/Source/platform/ScriptForbiddenScope.h:11:0,
                 from ../../third_party/WebKit/Source/bindings/core/v8/DOMWrapperMap.h:35,
                 from ../../third_party/WebKit/Source/bindings/core/v8/DOMDataStore.h:34,
                 from ../../third_party/WebKit/Source/bindings/core/v8/V8Binding.h:35,
                 from ../../third_party/WebKit/Source/bindings/core/v8/Dictionary.h:34,
                 from gen/blink/bindings/core/v8/UnionTypesCore.h:10,
                 from ../../third_party/WebKit/Source/core/events/EventTarget.h:36,
                 from ../../third_party/WebKit/Source/core/dom/Node.h:34,
                 from ../../third_party/WebKit/Source/core/dom/ContainerNode.h:29,
                 from ../../third_party/WebKit/Source/core/editing/Position.h:30,
                 from ../../third_party/WebKit/Source/core/editing/PositionWithAffinity.h:9,
                 from ../../third_party/WebKit/Source/core/layout/HitTestResult.h:26,
                 from ../../third_party/WebKit/Source/core/layout/HitTestCache.h:9,
                 from ../../third_party/WebKit/Source/core/layout/LayoutView.h:26,
                 from ../../third_party/WebKit/Source/core/layout/api/LayoutViewItem.h:9,
                 from ../../third_party/WebKit/Source/core/frame/FrameView.h:35,
                 from ../../third_party/WebKit/Source/core/frame/FrameView.cpp:27:
../../third_party/WebKit/Source/wtf/TemporaryChange.h:55:9: error: '*((void*)(& isUpdatingAllLifecyclePhasesScope)+8).WTF::TemporaryChange<bool>::m_scopedVariable' may be used uninitialized in this function [-Werror=maybe-uninitialized]
         m_scopedVariable = m_originalValue;
         ^
../../third_party/WebKit/Source/core/frame/FrameView.cpp:2404:37: note: '*((void*)(& isUpdatingAllLifecyclePhasesScope)+8).WTF::TemporaryChange<bool>::m_scopedVariable' was declared here
     Optional<TemporaryChange<bool>> isUpdatingAllLifecyclePhasesScope;
                                     ^
cc1plus: all warnings being treated as errors
ninja: build stopped: subcommand failed.


If the test is disabled, please downgrade to Pri-2.
 
Cc: esprehn@chromium.org jbudorick@chromium.org jbroman@chromium.org miguelg@chromium.org thakis@chromium.org
This is blocking perf tests on all devices with arm64, so I'm reverting the culprit.
Revert landed (https://codereview.chromium.org/1908903002/), so I'm lowering the priority of this bug.
Labels: -Pri-1 Pri-2

Comment 4 by danakj@chromium.org, Apr 21 2016

Status: Fixed (was: Untriaged)

Sign in to add a comment