Commit 208bda48 authored by Victor Gomes's avatar Victor Gomes Committed by V8 LUCI CQ

[bazel] Remove zlib dependency

That entails that V8 built with bazel does not support
snapshot compression.

NOTE: Snapshot compression has always been supported, but it has never
been enabled. We do not even have a flag for it.

Change-Id: I4b67db618be2c62e6a6f34e71eb0dac2ec26fa36
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3834031Reviewed-by: 's avatarCamillo Bruni <cbruni@chromium.org>
Commit-Queue: Victor Gomes <victorgomes@chromium.org>
Cr-Commit-Position: refs/heads/main@{#82537}
parent 45620273
......@@ -324,7 +324,6 @@ v8_config(
},
defines = [
"GOOGLE3",
"CHROMIUM_ZLIB_NO_CHROMECONF",
"ENABLE_DEBUGGER_SUPPORT",
"V8_ADVANCED_BIGINT_ALGORITHMS",
"V8_CONCURRENT_MARKING",
......@@ -2091,8 +2090,6 @@ filegroup(
"src/snapshot/shared-heap-deserializer.cc",
"src/snapshot/shared-heap-serializer.h",
"src/snapshot/shared-heap-serializer.cc",
"src/snapshot/snapshot-compression.cc",
"src/snapshot/snapshot-compression.h",
"src/snapshot/snapshot-data.cc",
"src/snapshot/snapshot-data.h",
"src/snapshot/snapshot-source-sink.cc",
......@@ -3575,8 +3572,6 @@ v8_library(
deps = [
":v8_libbase",
"//external:base_trace_event_common",
"//external:zlib",
"//external:zlib_compression_utils",
],
)
......
......@@ -34,22 +34,6 @@ pip_install(
requirements = "//:bazel/requirements.txt",
)
new_local_repository(
name = "com_googlesource_chromium_zlib",
build_file = "bazel/BUILD.zlib",
path = "third_party/zlib",
)
bind(
name = "zlib",
actual = "@com_googlesource_chromium_zlib//:zlib",
)
bind(
name = "zlib_compression_utils",
actual = "@com_googlesource_chromium_zlib//:zlib_compression_utils",
)
new_local_repository(
name = "com_googlesource_chromium_icu",
build_file = "bazel/BUILD.icu",
......
# Copyright 2021 the V8 project authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
cc_library(
name = "zlib",
srcs = [
"adler32.c",
"chromeconf.h",
"compress.c",
"contrib/optimizations/insert_string.h",
"cpu_features.c",
"cpu_features.h",
"crc32.c",
"crc32.h",
"deflate.c",
"deflate.h",
"gzclose.c",
"gzguts.h",
"gzlib.c",
"gzread.c",
"gzwrite.c",
"infback.c",
"inffast.c",
"inffast.h",
"inffixed.h",
"inflate.c",
"inflate.h",
"inftrees.c",
"inftrees.h",
"slide_hash_simd.h",
"trees.c",
"trees.h",
"uncompr.c",
"zconf.h",
"zlib.h",
"zutil.c",
"zutil.h",
],
hdrs = [
"zlib.h",
],
defines = [
"CHROMIUM_ZLIB_NO_CHROMECONF",
"CPU_NO_SIMD",
] + select({
"@platforms//os:windows": [],
"//conditions:default": [
"HAVE_HIDDEN",
],
}),
include_prefix = "third_party/zlib",
visibility = ["//visibility:public"],
)
cc_library(
name = "zlib_compression_utils",
srcs = [
"google/compression_utils_portable.cc",
],
hdrs = [
"google/compression_utils_portable.h",
],
include_prefix = "third_party/zlib",
visibility = ["//visibility:public"],
copts = ["-Wnodeprecated-non-prototype"],
deps = [
"//external:zlib",
],
)
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment