[Wconversion] Suppress glslang issue (KhronosGroup#2404)

cdotstout · shaybarak · web-flow · commit 1815d86a224b · 2020-09-26T11:02:33.000-06:00
The glslang public includes dir contains headers with implicit conversion issues. Add -Wno-conversion to glslang's public config. Bug: 60140 Bug: 58162 Change-Id: Iec27cb4242e9fdceddd6a3e02044a0bccfa0ce36 Reviewed-on: https://fuchsia-review.googlesource.com/c/third_party/glslang/+/429054 Reviewed-by: Petr Hosek <phosek@google.com> Co-authored-by: Shai Barack <shayba@google.com>
diff --git a/BUILD.gn b/BUILD.gn
@@ -76,6 +76,7 @@ if (!defined(glslang_angle)) {
 
 config("glslang_public") {
   include_dirs = [ "." ]
+  cflags = [ "-Wno-conversion" ]
 }
 
 config("glslang_hlsl") {

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ if (!defined(glslang_angle)) {`
`76`	`76`
`77`	`77`	`config("glslang_public") {`
`78`	`78`	`include_dirs = [ "." ]`
	`79`	`+ cflags = [ "-Wno-conversion" ]`
`79`	`80`	`}`
`80`	`81`
`81`	`82`	`config("glslang_hlsl") {`