New issue
Advanced search Search tips

Issue 920629 link

Starred by 1 user
Status: WontFix
Owner:
cblume@chromium.org
Closed: Jan 11
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Android compile fails with enable_vulkan=false

Project Member Reported by agrieve@chromium.org, Jan 10 
GN Args:

target_os = "android"
enable_vulkan = false
is_official_build = true  # Probably not required

ninja chrome_public_apk

Error is:
FAILED: obj/gpu/command_buffer/service/gles2_sources/shared_image_backing_factory_ahardwarebuffer.o
/usr/local/google/home/agrieve/goma/gomacc ../../third_party/llvm-build/Release+Asserts/bin/clang++ -MMD -MF obj/gpu/command_buffer/service/gles2_sources/shared_image_backing_factory_ahardwarebuffer.o.d -DVK_USE_PLATFORM_ANDROID_KHR -DNO_TCMALLOC -DSAFE_BROWSING_DB_REMOTE -DOFFICIAL_BUILD -DCHROMIUM_BUILD -D_GNU_SOURCE -DANDROID -DHAVE_SYS_UIO_H -DANDROID_NDK_VERSION_ROLL=r16_1 -DCR_CLANG_REVISION=\"349417-2\" -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D_FORTIFY_SOURCE=2 -D__GNU_SOURCE=1 -DCHROMIUM_CXX_TWEAK_INLINES -DNDEBUG -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 -DGPU_GLES2_IMPLEMENTATION -DRASTER_IMPLEMENTATION -DWEBGPU_IMPLEMENTATION -DWEBP_EXTERN=extern -DUSE_EGL -DGOOGLE_PROTOBUF_NO_RTTI -DGOOGLE_PROTOBUF_NO_STATIC_INITIALIZER -DHAVE_PTHREAD -DSK_HAS_PNG_LIBRARY -DSK_HAS_WEBP_LIBRARY -DSK_HAS_JPEG_LIBRARY -DSK_SUPPORT_GPU=1 -DSK_GPU_WORKAROUNDS_HEADER=\"gpu/config/gpu_driver_bug_workaround_autogen.h\" -DSK_BUILD_FOR_ANDROID -DUSE_CHROMIUM_SKIA -DU_USING_ICU_NAMESPACE=0 -DU_ENABLE_DYLOAD=0 -DUSE_CHROMIUM_ICU=1 -DU_STATIC_IMPLEMENTATION -DICU_UTIL_DATA_IMPL=ICU_UTIL_DATA_FILE -DUCHAR_TYPE=uint16_t -I../../third_party/mesa_headers -I../.. -Igen -I../../third_party/khronos -I../../gpu -I../../third_party/libwebp/src -I../../third_party/protobuf/src -Igen/protoc_out -I../../third_party/protobuf/src -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/docs -I../../third_party/skia/include/effects -I../../third_party/skia/include/encode -I../../third_party/skia/include/gpu -I../../third_party/skia/include/pathops -I../../third_party/skia/include/ports -I../../third_party/skia/include/utils -I../../third_party/skia/include/codec -I../../third_party/skia/src/gpu -I../../third_party/skia/src/sksl -I../../third_party/skia/modules/skottie/include -I../../third_party/angle/include -I../../third_party/angle/src -I../../third_party/angle/src/common/third_party/base -Igen/angle -I../../third_party/angle/include -I../../third_party/re2/src -I../../third_party/zlib -I../../third_party/icu/source/common -I../../third_party/icu/source/i18n -I../../third_party/ced/src -I../../third_party/mesa_headers -I../../third_party/libsync/src -I../../third_party/libsync/src/include -fprofile-sample-use=../../chrome/android/profiles/afdo.prof -fprofile-sample-accurate -fno-strict-aliasing --param=ssp-buffer-size=4 -fstack-protector -funwind-tables -fPIC -fcolor-diagnostics -fmerge-all-constants -Xclang -mllvm -Xclang -instcombine-lower-dbg-declare=0 -Xclang -fdebug-compilation-dir -Xclang . -no-canonical-prefixes -flto=thin -ffunction-sections -fno-short-enums --target=arm-linux-androideabi -isystem../../third_party/android_ndk/sysroot/usr/include/arm-linux-androideabi -D__ANDROID_API__=16 -DHAVE_PTHREAD_COND_TIMEDWAIT_MONOTONIC=1 -march=armv7-a -mfloat-abi=softfp -mtune=generic-armv7-a -mfpu=neon -mthumb -Wall -Werror -Wextra -Wimplicit-fallthrough -Wthread-safety -Wno-missing-field-initializers -Wno-unused-parameter -Wno-c++11-narrowing -Wno-unneeded-internal-declaration -Wno-undefined-var-template -Wno-ignored-pragma-optimize -fomit-frame-pointer -gdwarf-3 -g1 -fdebug-info-for-profiling -fvisibility=hidden -Xclang -add-plugin -Xclang find-bad-constructs -Xclang -plugin-arg-find-bad-constructs -Xclang check-ipc -Wheader-hygiene -Wstring-conversion -Wtautological-overlap-compare -Wexit-time-destructors -Wexit-time-destructors -Wexit-time-destructors -O2 -fno-ident -fdata-sections -ffunction-sections -std=c++14 -fno-exceptions -fno-rtti --sysroot=../../third_party/android_ndk/sysroot -nostdinc++ -isystem../../third_party/android_ndk/sources/cxx-stl/llvm-libc++/include -isystem../../third_party/android_ndk/sources/cxx-stl/llvm-libc++abi/include -isystem../../third_party/android_ndk/sources/android/support/include -fvisibility-inlines-hidden -c ../../gpu/command_buffer/service/shared_image_backing_factory_ahardwarebuffer.cc -o obj/gpu/command_buffer/service/gles2_sources/shared_image_backing_factory_ahardwarebuffer.o
In file included from ../../gpu/command_buffer/service/shared_image_backing_factory_ahardwarebuffer.cc:27:
In file included from ../../gpu/vulkan/vulkan_device_queue.h:8:
../../third_party/android_ndk/sysroot/usr/include/vulkan/vulkan.h:114:14: error: redefinition of 'VkPipelineCacheHeaderVersion'
typedef enum VkPipelineCacheHeaderVersion {
             ^
../../third_party/skia/include/c/../../include/third_party/vulkan/vulkan/vulkan_core.h:114:14: note: previous definition is here
typedef enum VkPipelineCacheHeaderVersion {
             ^
In file included from ../../gpu/command_buffer/service/shared_image_backing_factory_ahardwarebuffer.cc:27:
In file included from ../../gpu/vulkan/vulkan_device_queue.h:8:

Comment 1 by agrieve@chromium.org, Jan 10

Description: Show this description

Comment 2 by cblume@chromium.org, Jan 11

Status: WontFix (was: Assigned)
I think we can not worry about enable_vulkan=false on Android. Our next release should have it enabled.

I promised you I would do another binary size analysis when this day came. I'll do that soon, but on the original bug. :)
►

Sign in to add a comment