v8/test/cctest jumbo compilation error on Arm - too many F5 typedefs |
||
Issue descriptionhttps://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Ftryserver.chromium.android%2Fandroid_clang_dbg_recipe%2F357754%2F%2B%2Frecipes%2Fsteps%2Fcompile__with_patch_%2F0%2Fstdout FAILED: obj/v8/test/cctest/cctest_sources/cctest_sources_jumbo_2.o /b/c/goma_client/gomacc ../../third_party/llvm-build/Release+Asserts/bin/clang++ -MMD -MF obj/v8/test/cctest/cctest_sources/cctest_sources_jumbo_2.o.d -DBUILDING_V8_SHARED -DV8_INTL_SUPPORT -DV8_DEPRECATION_WARNINGS -DNO_TCMALLOC -DMEMORY_TOOL_REPLACES_ALLOCATOR -DMEMORY_SANITIZER_INITIAL_SIZE -DADDRESS_SANITIZER -DSAFE_BROWSING_DB_REMOTE -DCHROMIUM_BUILD -DFIELDTRIAL_TESTING_ENABLED -DCR_CLANG_REVISION=\"313786-1\" -D_FILE_OFFSET_BITS=64 -DANDROID -DHAVE_SYS_UIO_H -DANDROID_NDK_VERSION_ROLL=r12b_1 -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -DCOMPONENT_BUILD -D__GNU_SOURCE=1 -DCHROMIUM_CXX_TWEAK_INLINES -D__compiler_offsetof=__builtin_offsetof -Dnan=__builtin_nan -D_DEBUG -DDYNAMIC_ANNOTATIONS_ENABLED=1 -DWTF_USE_DYNAMIC_ANNOTATIONS=1 -DUSING_V8_SHARED -DV8_ENABLE_CHECKS -DENABLE_DISASSEMBLER -DV8_TYPED_ARRAY_MAX_SIZE_IN_HEAP=64 -DOBJECT_PRINT -DVERIFY_HEAP -DV8_TRACE_MAPS -DV8_ENABLE_CHECKS -DV8_INTL_SUPPORT -DENABLE_HANDLE_ZAPPING -DV8_USE_SNAPSHOT -DV8_USE_EXTERNAL_STARTUP_DATA -DV8_CONCURRENT_MARKING -DV8_CSA_WRITE_BARRIER -DV8_TARGET_ARCH_ARM -DCAN_USE_ARMV7_INSTRUCTIONS -DCAN_USE_VFP3_INSTRUCTIONS -DCAN_USE_VFP32DREGS -DCAN_USE_NEON -DDEBUG -DUSING_V8_BASE_SHARED -DUSING_V8_PLATFORM_SHARED -I../.. -Igen -I../../v8/include -Igen/v8/include -I../../v8 -I../../v8/include -ffp-contract=off -fno-strict-aliasing --param=ssp-buffer-size=4 -fstack-protector -Wno-builtin-macro-redefined -D__DATE__= -D__TIME__= -D__TIMESTAMP__= -funwind-tables -fPIC -pipe -fcolor-diagnostics -ffunction-sections -fno-short-enums --target=arm-linux-androideabi -march=armv7-a -mfloat-abi=softfp -mtune=generic-armv7-a -mfpu=neon -mthumb -Wall -Werror -Wextra -Wno-missing-field-initializers -Wno-unused-parameter -Wno-c++11-narrowing -Wno-covered-switch-default -Wno-unneeded-internal-declaration -Wno-inconsistent-missing-override -Wno-undefined-var-template -Wno-nonportable-include-path -Wno-address-of-packed-member -Wno-unused-lambda-capture -Wno-user-defined-warnings -Wno-enum-compare-switch -Wno-tautological-unsigned-zero-compare -Wno-null-pointer-arithmetic -Wno-tautological-unsigned-enum-zero-compare -fomit-frame-pointer -gdwarf-3 -g1 -fdebug-info-for-profiling -gline-tables-only -gcolumn-info -fno-omit-frame-pointer -fsanitize=address -fsanitize-address-use-after-scope -fsanitize-blacklist=../../tools/memory/asan/blacklist.txt -fvisibility=hidden -Xclang -load -Xclang ../../third_party/llvm-build/Release+Asserts/lib/libFindBadConstructs.so -Xclang -add-plugin -Xclang find-bad-constructs -Xclang -plugin-arg-find-bad-constructs -Xclang check-auto-raw-pointer -Xclang -plugin-arg-find-bad-constructs -Xclang check-ipc -Wheader-hygiene -Wstring-conversion -Wtautological-overlap-compare -Wsign-compare -Winconsistent-missing-override -O2 -fno-ident -fdata-sections -ffunction-sections -Wno-undefined-bool-conversion -Wno-tautological-undefined-compare -std=gnu++14 -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 --sysroot=../../third_party/android_tools/ndk/platforms/android-16/arch-arm -fno-exceptions -fvisibility-inlines-hidden -c gen/out/Debug/gen/v8/test/cctest/cctest_sources_jumbo_2.cc -o obj/v8/test/cctest/cctest_sources/cctest_sources_jumbo_2.o In file included from gen/out/Debug/gen/v8/test/cctest/cctest_sources_jumbo_2.cc:86: gen/out/Debug/gen/v8/test/cctest/../../../../../../../../../v8/test/cctest/test-macro-assembler-arm.cc:45:15: error: typedef redefinition with different types ('int (*)(void *, void *, void *, void *, void *)' vs 'v8::internal::Object *(*)(uint32_t, void *, void *, int, int)' (aka 'v8::internal::Object *(*)(unsigned int, void *, void *, int, int)')) typedef int (*F5)(void*, void*, void*, void*, void*); ^ gen/out/Debug/gen/v8/test/cctest/../../../../../../../../../v8/test/cctest/test-assembler-arm.cc:51:19: note: previous definition is here typedef Object* (*F5)(uint32_t p0, void* p1, void* p2, int p3, int p4); ^ 1 error generated.
,
Oct 3 2017
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/0c93ef357d7e0e969322fe0378463203af9358a4 commit 0c93ef357d7e0e969322fe0378463203af9358a4 Author: Mostyn Bramley-Moore <mostynb@vewd.com> Date: Tue Oct 03 06:24:30 2017 [jumbo] arm unittests fixup Avoid F5 symbol clash. Bug: chromium:770684 Change-Id: I9e851c1fe5abbdc4d032b7efdd0462c96d0c8ef3 Reviewed-on: https://chromium-review.googlesource.com/696063 Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Cr-Commit-Position: refs/heads/master@{#48267} [modify] https://crrev.com/0c93ef357d7e0e969322fe0378463203af9358a4/test/cctest/test-assembler-arm.cc [modify] https://crrev.com/0c93ef357d7e0e969322fe0378463203af9358a4/test/cctest/test-macro-assembler-arm.cc
,
Oct 3 2017
,
Oct 19 2017
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/2551f73eebc51e4f250a434e5345894106b618fb commit 2551f73eebc51e4f250a434e5345894106b618fb Author: Mostyn Bramley-Moore <mostynb@vewd.com> Date: Thu Oct 19 09:15:44 2017 [jumbo] fix another set of unittest compilation errors This makes jumbo_file_merge_limit=50 work again. Bug: chromium:770684 Change-Id: I5db6566da876d71ea6ba50ff03b7652074b0a35f Reviewed-on: https://chromium-review.googlesource.com/725818 Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Cr-Commit-Position: refs/heads/master@{#48728} [modify] https://crrev.com/2551f73eebc51e4f250a434e5345894106b618fb/src/interpreter/bytecode-array-writer.h [modify] https://crrev.com/2551f73eebc51e4f250a434e5345894106b618fb/test/unittests/compiler/machine-operator-unittest.cc [modify] https://crrev.com/2551f73eebc51e4f250a434e5345894106b618fb/test/unittests/compiler/node-unittest.cc [modify] https://crrev.com/2551f73eebc51e4f250a434e5345894106b618fb/test/unittests/compiler/simplified-operator-unittest.cc [modify] https://crrev.com/2551f73eebc51e4f250a434e5345894106b618fb/test/unittests/compiler/value-numbering-reducer-unittest.cc [modify] https://crrev.com/2551f73eebc51e4f250a434e5345894106b618fb/test/unittests/interpreter/bytecode-array-writer-unittest.cc [modify] https://crrev.com/2551f73eebc51e4f250a434e5345894106b618fb/test/unittests/interpreter/interpreter-assembler-unittest.cc [modify] https://crrev.com/2551f73eebc51e4f250a434e5345894106b618fb/test/unittests/interpreter/interpreter-assembler-unittest.h
,
Oct 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/26ac49bfb319ecf7a2f3e0a2fe08046c26fb52b1 commit 26ac49bfb319ecf7a2f3e0a2fe08046c26fb52b1 Author: Mostyn Bramley-Moore <mostynb@vewd.com> Date: Fri Oct 20 12:43:57 2017 [jumbo] fix android/windows kDummyOperator collision Bug: chromium:770684 Change-Id: I4e4efc10dad1d3bb438fddc74098b36a6b9e1054 Reviewed-on: https://chromium-review.googlesource.com/730203 Reviewed-by: Clemens Hammacher <clemensh@chromium.org> Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Cr-Commit-Position: refs/heads/master@{#48787} [modify] https://crrev.com/26ac49bfb319ecf7a2f3e0a2fe08046c26fb52b1/test/unittests/compiler/graph-unittest.cc [modify] https://crrev.com/26ac49bfb319ecf7a2f3e0a2fe08046c26fb52b1/test/unittests/compiler/schedule-unittest.cc |
||
►
Sign in to add a comment |
||
Comment 1 by most...@vewd.com
, Oct 2 2017Status: Started (was: Untriaged)