encounter
/
dawn-cmake
mirror of https://github.com/encounter/dawn-cmake.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

GN: Rely on the Chromium macOS deployment target again. 

Chromium requires macOS 10.11 so we no longer need to special case this
in Dawn for Metal support anymore.

Bug: None
Change-Id: Ib5dee5464a3ec27abdd0f5c62e51caa42171fadb
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/88303
Auto-Submit: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Austin Eng <enga@chromium.org>
This commit is contained in:
Corentin Wallez 2022-04-28 15:35:13 +00:00 committed by Dawn LUCI CQ
parent 0da5e8b35d
commit e8592a4913
2 changed files with 0 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
.gn
View File

@ -20,13 +20,6 @@ script_executable = "python3"
default_args = { default_args = {
clang_use_chrome_plugins = false clang_use_chrome_plugins = false
# Override the mac version so standalone Dawn compiles with at least 10.11
# which allows us to not skip the -Wunguarded-availability warning and get
# proper warnings for use of APIs that are 10.12 and above (even if
# Chromium is still on 10.10).
mac_deployment_target = "10.11.0"
mac_min_system_version = "10.11.0"
angle_enable_abseil = false angle_enable_abseil = false
angle_standalone = false angle_standalone = false
angle_build_all = false angle_build_all = false

23
src/dawn/native/BUILD.gn
View File

@ -19,15 +19,6 @@ import("${dawn_root}/generator/dawn_generator.gni")
import("${dawn_root}/scripts/dawn_component.gni") import("${dawn_root}/scripts/dawn_component.gni")
import("${dawn_root}/scripts/dawn_features.gni") import("${dawn_root}/scripts/dawn_features.gni")
# Import mac_deployment_target
if (is_mac) {
if (dawn_has_build) {
import("//build/config/mac/mac_sdk.gni")
} else {
mac_deployment_target = "10.11.0"
}
}
# The VVLs are an optional dependency, only use it if the path has been set. # The VVLs are an optional dependency, only use it if the path has been set.
enable_vulkan_validation_layers = dawn_enable_vulkan_validation_layers && enable_vulkan_validation_layers = dawn_enable_vulkan_validation_layers &&
dawn_vulkan_validation_layers_dir != "" dawn_vulkan_validation_layers_dir != ""
@ -66,20 +57,6 @@ group("abseil") {
config("internal") { config("internal") {
configs = [ "${dawn_root}/src/dawn/common:internal_config" ] configs = [ "${dawn_root}/src/dawn/common:internal_config" ]
# Suppress warnings that Metal isn't in the deployment target of Chrome:
# initialization of the Metal backend is behind a IsMetalSupported check so
# Dawn won't call Metal functions on macOS 10.10.
# At the time this is written Chromium supports 10.10.0 and above, so if we
# aren't on 10.11 it means we are on 10.11 and above, and Metal is available.
# Skipping this check on 10.11 and above is important as it allows getting
# proper compilation warning when using 10.12 and above feature for example.
# TODO(crbug.com/1004024): Consider using API_AVAILABLE annotations on all
# metal code in dawn once crbug.com/1004024 is sorted out if Chromium still
# supports 10.10 then.
if (is_mac && mac_deployment_target == "10.10.0") {
cflags_objcc = [ "-Wno-unguarded-availability" ]
}
} }
config("weak_framework") { config("weak_framework") {