diff --git a/config/android/rules.gni b/config/android/rules.gni index 8582d47fa..8ca064248 100644 --- a/config/android/rules.gni +++ b/config/android/rules.gni @@ -74,6 +74,12 @@ template("generate_jni") { rebase_path(jni_generator_jarjar_file, root_build_dir), ] } + if (!is_clang) { + # Clang builds currently fail with --native_exports_optional due to + # http://llvm.org/bugs/show_bug.cgi?id=22602 - only enable for gcc. + # http://crbug.com/442327 + args += [ "--native_exports_optional" ] + } } config("jni_includes_${target_name}") { @@ -184,6 +190,12 @@ template("generate_jar_jni") { "--includes", rebase_path(jni_generator_include, root_build_dir), ] + if (!is_clang) { + # Clang builds currently fail with --native_exports_optional due to + # http://llvm.org/bugs/show_bug.cgi?id=22602 - only enable for gcc. + # http://crbug.com/442327 + args += [ "--native_exports_optional" ] + } } } diff --git a/jar_file_jni_generator.gypi b/jar_file_jni_generator.gypi index 4c01c8a02..9472c10be 100644 --- a/jar_file_jni_generator.gypi +++ b/jar_file_jni_generator.gypi @@ -73,5 +73,13 @@ '<(DEPTH)/build/android/android_exports.gyp:android_exports', ], }], + ['clang==0', { + # Clang builds currently fail with --native_exports_optional due to + # http://llvm.org/bugs/show_bug.cgi?id=22602 - only enable for gcc. + # http://crbug.com/442327 + 'variables': { + 'native_exports%': '--native_exports_optional', + }, + }], ], } diff --git a/jni_generator.gypi b/jni_generator.gypi index 6edc512ec..853b5f6e7 100644 --- a/jni_generator.gypi +++ b/jni_generator.gypi @@ -92,6 +92,14 @@ '<(DEPTH)/build/android/android_exports.gyp:android_exports', ], }], + ['clang==0', { + # Clang builds currently fail with --native_exports_optional due to + # http://llvm.org/bugs/show_bug.cgi?id=22602 - only enable for gcc. + # http://crbug.com/442327 + 'variables': { + 'native_exports%': '--native_exports_optional', + }, + }], ], }