Commit 2f71802b by Jamie Madill Committed by Commit Bot

GN: Don't expose glslang when unused.

The ninja generation step could get confused if we try to build the glslang wrapper project on platforms that don't support glslang. Turn it off if the Metal / Vulkan back-ends are not enabled. Bug: angleproject:4053 Change-Id: If4dd2856681f18f4f45e952bf88e33371b9a1e3c Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/1884010Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org> Reviewed-by: 's avatarJonah Ryan-Davis <jonahr@google.com> Commit-Queue: Jamie Madill <jmadill@chromium.org>
parent a12b3032
...@@ -595,7 +595,9 @@ angle_source_set("libANGLE_headers") { ...@@ -595,7 +595,9 @@ angle_source_set("libANGLE_headers") {
] ]
} }
angle_source_set("angle_glslang_wrapper") { # Don't even try to build glslang if Metal and Vulkan are disabled.
if (angle_enable_vulkan || angle_enable_metal) {
angle_source_set("angle_glslang_wrapper") {
sources = [ sources = [
"src/libANGLE/renderer/glslang_wrapper_utils.cpp", "src/libANGLE/renderer/glslang_wrapper_utils.cpp",
"src/libANGLE/renderer/glslang_wrapper_utils.h", "src/libANGLE/renderer/glslang_wrapper_utils.h",
...@@ -605,6 +607,7 @@ angle_source_set("angle_glslang_wrapper") { ...@@ -605,6 +607,7 @@ angle_source_set("angle_glslang_wrapper") {
"${angle_glslang_dir}:glslang_default_resource_limits_sources", "${angle_glslang_dir}:glslang_default_resource_limits_sources",
"${angle_glslang_dir}:glslang_sources", "${angle_glslang_dir}:glslang_sources",
] ]
}
} }
angle_source_set("libANGLE_base") { angle_source_set("libANGLE_base") {
......
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