New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
7.3-lkgr #44
7.3-lkgr #44
Conversation
30664b9
to
1ce564b
Compare
android.patch
Outdated
'mixins': { | ||
'android': { | ||
- 'gn_args': 'target_os="android" v8_android_log_stdout=true', | ||
+ 'gn_args': 'target_os="android" v8_android_log_stdout=false', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Set v8_android_log_stdout
to false
as we want to output using __android_log
@@ -8,10 +8,10 @@ def build(scm, arch, mode) { | |||
def expectedLibraries = ['monolith'] | |||
def labels = 'ninja && git && android-ndk && android-sdk && python' | |||
if (arch.equals('ia32') || arch.equals('arm')) { | |||
labels += ' && (linux || (osx && xcode-9))' // Need xcode-9 or older on mac, as 32-bit x86 was removed in xcode 10 | |||
labels += ' && (osx && xcode-9)' // Need xcode-9 or older on mac, as 32-bit x86 was removed in xcode 10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only build on osx
nodes as we want a macOS compatible mksnapshot
5258d5a
to
f5e6c43
Compare
common_optimize_on_cflags += [ | ||
# Put data in separate COMDATs. This allows the linker | ||
# to put bit-identical constants at the same address even if | ||
# they're unrelated constants, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use -Ofast
instead of -O3
"-Wl,--exclude-libs=libvpx_assembly_arm.a", | ||
] | ||
|
||
# libc++ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Specify NDK libc++ headers
@@ -44,7 +44,9 @@ template("android_clang_toolchain") { | |||
# The tools should be run relative to the build dir. | |||
_tool_prefix = rebase_path("$_android_tool_prefix", root_build_dir) | |||
|
|||
_prefix = rebase_path("$clang_base_path/bin", root_build_dir) | |||
# Use android ndk clang | |||
_prefix = "$android_ndk_root/toolchains/llvm/prebuilt/darwin-x86_64/bin" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use NDK clang instead of pre-built dependency
770d949
to
47ebdb1
Compare
1130db5
to
e075e81
Compare
so I see you pegged us to osx nodes - does this mean the mksnapshot will only work on Mac? What about Windows? |
V8 7.3.492.26
mksnapshot
to create memory snapshots