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

797 lines
31 KiB
Plaintext
Raw Normal View History

Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
# Copyright 2021 The Tint Authors
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
import("//build_overrides/build.gni")
import("//testing/test.gni")
import("../tint_overrides_with_defaults.gni")
###############################################################################
# Gtest Gmock - Handle building inside and outside of Chromium.
###############################################################################
# When building outside of Chromium we need to define our own targets for GTest
# and GMock. However when compiling inside of Chromium we need to reuse the
# existing targets, both because Chromium has a special harness for swarming
# and because otherwise the "gn check" fails.
if (!build_with_chromium) {
# When we aren't in Chromium we define out own targets based on the location
# of the googletest repo.
config("gtest_config") {
include_dirs = [
"${tint_googletest_dir}/googletest",
"${tint_googletest_dir}/googletest/include",
]
}
static_library("gtest") {
testonly = true
sources = [ "${tint_googletest_dir}/googletest/src/gtest-all.cc" ]
public_configs = [ ":gtest_config" ]
}
config("gmock_config") {
include_dirs = [
"${tint_googletest_dir}/googlemock",
"${tint_googletest_dir}/googlemock/include",
"${tint_googletest_dir}/googletest/include",
]
}
static_library("gmock") {
testonly = true
sources = [ "${tint_googletest_dir}/googlemock/src/gmock-all.cc" ]
public_configs = [ ":gmock_config" ]
}
group("gmock_and_gtest") {
testonly = true
public_deps = [
":gmock",
":gtest",
]
}
} else {
# When we are in Chromium we reuse its targets, and also add some deps that
# are needed to launch the test in swarming mode.
group("gmock_and_gtest") {
testonly = true
public_deps = [
"//base",
"//base/test:test_support",
"//testing/gmock",
"//testing/gtest",
"//third_party/googletest:gmock",
]
}
}
###############################################################################
# Wrapping of Chromium targets
###############################################################################
# These targets are separated because they are Chromium sources files that
# can't use the tint_internal config, otherwise Tint's warning flags get
# applied while compiling a bunch of Chromium's //base (via header inclusion)
source_set("tint_unittests_main") {
testonly = true
deps = [ ":gmock_and_gtest" ]
if (build_with_chromium) {
sources = [ "//gpu/tint_unittests_main.cc" ]
} else {
sources = [ "../src/test_main.cc" ]
Add --dump-spirv option to tint_unittests The --dump-spirv option tells tint_unittests to output the SPIR-V assembly text for a module which did not make the SPIR-V reader fail. This lets us get extract a corpus of SPIR-V modules, and lets us more easily verify that the test shaders are valid in the first place. Also: - Add test/extract-spvasm.py to split that output to separate SPIR-V assembly files - Add optional second argument test/test-all.sh to specify a directory look for input files. - BUILD.gn: Add dependency from //test:tint_unittests_main to //test:tint_unittests_config to pick up source dependency on the internal header of the SPIRV-Tools optimizer, needed by the indirection through src/reader/spirv/parser_impl_test_helper.h This is useful for bulk testing Fixed: tint:756 Change-Id: I4fe232ac736003f7d9be35544328302d652381ea Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49605 Auto-Submit: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-05-05 09:46:31 +00:00
configs += [ ":tint_unittests_config" ]
Get `gn gen --check` actually working in repo Fixes an issue in the .gn that allows the check to run as expected. As well as fixes issues discovered from running the check. BUG=tint:735 Change-Id: I48cd01339d06132933c3da4f0e9d45cdbaff3b55 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48700 Commit-Queue: Ryan Harrison <rharrison@chromium.org> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org>
2021-04-21 19:05:14 +00:00
deps += [
":tint_unittests_hlsl_writer_src",
":tint_unittests_msl_writer_src",
Add --dump-spirv option to tint_unittests The --dump-spirv option tells tint_unittests to output the SPIR-V assembly text for a module which did not make the SPIR-V reader fail. This lets us get extract a corpus of SPIR-V modules, and lets us more easily verify that the test shaders are valid in the first place. Also: - Add test/extract-spvasm.py to split that output to separate SPIR-V assembly files - Add optional second argument test/test-all.sh to specify a directory look for input files. - BUILD.gn: Add dependency from //test:tint_unittests_main to //test:tint_unittests_config to pick up source dependency on the internal header of the SPIRV-Tools optimizer, needed by the indirection through src/reader/spirv/parser_impl_test_helper.h This is useful for bulk testing Fixed: tint:756 Change-Id: I4fe232ac736003f7d9be35544328302d652381ea Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49605 Auto-Submit: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-05-05 09:46:31 +00:00
":tint_unittests_spv_reader_src",
Get `gn gen --check` actually working in repo Fixes an issue in the .gn that allows the check to run as expected. As well as fixes issues discovered from running the check. BUG=tint:735 Change-Id: I48cd01339d06132933c3da4f0e9d45cdbaff3b55 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48700 Commit-Queue: Ryan Harrison <rharrison@chromium.org> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org>
2021-04-21 19:05:14 +00:00
"${tint_root_dir}/src:libtint",
]
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
}
}
###############################################################################
# Tests - For libtint core and optional modules
###############################################################################
config("tint_unittests_config") {
include_dirs = [
"${tint_googletest_dir}/googlemock/include",
"${tint_googletest_dir}/googletest/include",
]
configs = [
"${tint_root_dir}/src:tint_common_config",
"${tint_root_dir}/src:tint_public_config",
]
}
Rename sem_*.cc to *.cc This makes the .cc files match the header file name, which allows for the text editor "header flip" feature to actually work. The reason these files were named this way was because GN doesn't allow name conflicts in the same source set, despite the files being in different directories. This change splits the files into different source sets. To do so, we use GN templates, which also reduces duplication in each target definition. Bug: tint:724 Change-Id: I9a7ed3912e4b85b2b38d360805203f3488b86c4c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-04-19 15:20:03 +00:00
template("tint_unittests_source_set") {
source_set(target_name) {
forward_variables_from(invoker, "*", [ "configs" ])
if (defined(invoker.configs)) {
configs += invoker.configs
}
configs += [ ":tint_unittests_config" ]
if (build_with_chromium) {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [ "//build/config/compiler:no_chromium_code" ]
}
testonly = true
if (!defined(invoker.deps)) {
deps = []
}
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
deps += [
":gmock_and_gtest",
"${tint_root_dir}/src:libtint",
test: Remove HLSL & MSL Validate() logic We have the end-to-end test-runner which validates all this stuff. There's no need to also Validate in the unit tests. Change-Id: I8fbc1ecd395efcedec4aa41085e691c88f3b66a5 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/54160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-06-11 13:22:27 +00:00
"${tint_root_dir}/src:tint_utils_io",
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
]
Rename sem_*.cc to *.cc This makes the .cc files match the header file name, which allows for the text editor "header flip" feature to actually work. The reason these files were named this way was because GN doesn't allow name conflicts in the same source set, despite the files being in different directories. This change splits the files into different source sets. To do so, we use GN templates, which also reduces duplication in each target definition. Bug: tint:724 Change-Id: I9a7ed3912e4b85b2b38d360805203f3488b86c4c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-04-19 15:20:03 +00:00
}
}
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
tint_unittests_source_set("tint_unittests_ast_src") {
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
sources = [
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/alias_test.cc",
"../src/ast/array_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/assignment_statement_test.cc",
ast: Add atomic type Bug: tint:892 Change-Id: I00ab3e819a60762ff72b58f8a8255e56adadd71f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/54644 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-06-17 15:48:39 +00:00
"../src/ast/atomic_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/binary_expression_test.cc",
"../src/ast/binding_decoration_test.cc",
"../src/ast/bitcast_expression_test.cc",
"../src/ast/block_statement_test.cc",
ast: Add 'Expression' suffix to literals Literals are now expressions, so in keeping with all the other expression types, suffix the class name with Expression. I'm not overly keen on requiring everything to have an Expression suffix, but consistency is better than personal preference. Bug: tint:888 Change-Id: Ida1f1b98c71d5d0fe6399bca938010cd5d3a00c3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/68840 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-11-09 09:35:00 +00:00
"../src/ast/bool_literal_expression_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/bool_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/break_statement_test.cc",
"../src/ast/builtin_decoration_test.cc",
"../src/ast/call_expression_test.cc",
"../src/ast/call_statement_test.cc",
"../src/ast/case_statement_test.cc",
"../src/ast/continue_statement_test.cc",
Implement texture_depth_multisampled_2d Implemented for all readers and writers. Cleaned up some verbose code in sem::Function and the Inspector in the process. Fixed: tint:1032 Change-Id: Ia6f2f59e6d2e511c89160b97be990e8b7c9828d9 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/59664 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com> Reviewed-by: James Price <jrprice@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: Ben Clayton <bclayton@google.com>
2021-07-26 22:19:48 +00:00
"../src/ast/depth_multisampled_texture_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/depth_texture_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/discard_statement_test.cc",
"../src/ast/else_statement_test.cc",
Add ast::ExternalTexture Also create it in wgsl parser. With this, we now create all the AST type nodes in the wgsl parser. Bug: tint:724 Change-Id: I575c9eb0ffbd60c3e7aca0b00227d9833ef65d58 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48962 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-04-26 14:55:25 +00:00
"../src/ast/external_texture_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/f32_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/fallthrough_statement_test.cc",
ast: Add 'Expression' suffix to literals Literals are now expressions, so in keeping with all the other expression types, suffix the class name with Expression. I'm not overly keen on requiring everything to have an Expression suffix, but consistency is better than personal preference. Bug: tint:888 Change-Id: Ida1f1b98c71d5d0fe6399bca938010cd5d3a00c3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/68840 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-11-09 09:35:00 +00:00
"../src/ast/float_literal_expression_test.cc",
ast: Add ForLoopStatement node Currently entirely unused. Bug: tint:952 Change-Id: I3ba2823024b81e927d8fe8a80fac4f82c02eac3e Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56761 Kokoro: Kokoro <noreply+kokoro@google.com> Auto-Submit: Ben Clayton <bclayton@google.com> Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-07-02 19:27:42 +00:00
"../src/ast/for_loop_statement_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/function_test.cc",
"../src/ast/group_decoration_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/i32_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/identifier_expression_test.cc",
"../src/ast/if_statement_test.cc",
Add tests for fuzzers::RandomGenerator BUG=tint:1019 Change-Id: Ia462080877a97348c5589bfa71231a832a7ebfd3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70081 Auto-Submit: Ryan Harrison <rharrison@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-24 14:55:05 +00:00
"../src/ast/index_accessor_expression_test.cc",
ast: Add 'Expression' suffix to literals Literals are now expressions, so in keeping with all the other expression types, suffix the class name with Expression. I'm not overly keen on requiring everything to have an Expression suffix, but consistency is better than personal preference. Bug: tint:888 Change-Id: Ida1f1b98c71d5d0fe6399bca938010cd5d3a00c3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/68840 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-11-09 09:35:00 +00:00
"../src/ast/int_literal_expression_test.cc",
ast: Add InterpolateDecoration and related enums Bug: tint:746 Change-Id: Ifd0de8b0e4400a153e67a8e301a7d0523e75898a Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56240 Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: James Price <jrprice@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-06-28 18:20:23 +00:00
"../src/ast/interpolate_decoration_test.cc",
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
"../src/ast/intrinsic_texture_helper_test.cc",
"../src/ast/intrinsic_texture_helper_test.h",
ast: Add InvariantDecoration Bug: tint:772 Change-Id: Icd1892a1affed10af8e7b90257129fa54360815e Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/57640 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Ben Clayton <bclayton@google.com>
2021-07-12 11:44:51 +00:00
"../src/ast/invariant_decoration_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/location_decoration_test.cc",
"../src/ast/loop_statement_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/matrix_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/member_accessor_expression_test.cc",
"../src/ast/module_clone_test.cc",
"../src/ast/module_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/multisampled_texture_test.cc",
Rename builtin constant_id to override There are no downstream usages, so we can skip deprecation. Allowing the ID to be omitted will be done in a separate patch. Fixed: tint:754 Change-Id: I3bd6de4d0f426fc3c66708bfd5b411a4051b375b Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49581 Commit-Queue: James Price <jrprice@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-04-30 17:14:19 +00:00
"../src/ast/override_decoration_test.cc",
ast: Add PhonyExpression Represents the `_` in a phony assignment `_ = <expr>` Bug: tint:1213 Change-Id: Ifb070277b7872aa5ae405e467e28191e625c4c5a Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/67062 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: David Neto <dneto@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-10-21 18:33:14 +00:00
"../src/ast/phony_expression_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/pointer_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/return_statement_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/sampled_texture_test.cc",
"../src/ast/sampler_test.cc",
ast: Add 'Expression' suffix to literals Literals are now expressions, so in keeping with all the other expression types, suffix the class name with Expression. I'm not overly keen on requiring everything to have an Expression suffix, but consistency is better than personal preference. Bug: tint:888 Change-Id: Ida1f1b98c71d5d0fe6399bca938010cd5d3a00c3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/68840 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-11-09 09:35:00 +00:00
"../src/ast/sint_literal_expression_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/stage_decoration_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/storage_texture_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/stride_decoration_test.cc",
Implement Default Struct Layout Implements https://github.com/gpuweb/gpuweb/pull/1447 SPIR-V Reader is still TODO, but continues to function as the offset decoration is still supported. Bug: tint:626 Bug: tint:629 Change-Id: Id574eb3a5c6729559382812de37b23f0c68fd406 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/43640 Commit-Queue: Ben Clayton <bclayton@chromium.org> Reviewed-by: David Neto <dneto@google.com>
2021-03-15 10:43:11 +00:00
"../src/ast/struct_member_align_decoration_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/struct_member_offset_decoration_test.cc",
Implement Default Struct Layout Implements https://github.com/gpuweb/gpuweb/pull/1447 SPIR-V Reader is still TODO, but continues to function as the offset decoration is still supported. Bug: tint:626 Bug: tint:629 Change-Id: Id574eb3a5c6729559382812de37b23f0c68fd406 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/43640 Commit-Queue: Ben Clayton <bclayton@chromium.org> Reviewed-by: David Neto <dneto@google.com>
2021-03-15 10:43:11 +00:00
"../src/ast/struct_member_size_decoration_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/struct_member_test.cc",
"../src/ast/struct_test.cc",
"../src/ast/switch_statement_test.cc",
"../src/ast/test_helper.h",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/texture_test.cc",
ast: Add TraverseExpressions() An ast::Expression traversal helper extracted from Resolver. Change-Id: I88754cbc86cc12cbf8348fb36a3f038904017f3d Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/67202 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-10-21 20:38:54 +00:00
"../src/ast/traverse_expressions_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/u32_test.cc",
ast: Add 'Expression' suffix to literals Literals are now expressions, so in keeping with all the other expression types, suffix the class name with Expression. I'm not overly keen on requiring everything to have an Expression suffix, but consistency is better than personal preference. Bug: tint:888 Change-Id: Ida1f1b98c71d5d0fe6399bca938010cd5d3a00c3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/68840 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-11-09 09:35:00 +00:00
"../src/ast/uint_literal_expression_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/unary_op_expression_test.cc",
"../src/ast/variable_decl_statement_test.cc",
"../src/ast/variable_test.cc",
ast: Add type nodes Copy all of the type classes from src/type into ast. Required the merging of: * type::Struct into the existing ast::Struct - ast::Struct now has a name. * type::AccessControl into the existing ast::AccessControl enumerator - The old ast::AccessControl enumerator is now ast::AccessControl::Access Bug: tint:724 Change-Id: Ibb950036ed551ec769c6d3d2c8fb411809cf6931 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48383 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-20 15:04:21 +00:00
"../src/ast/vector_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/ast/workgroup_decoration_test.cc",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
]
}
tint_unittests_source_set("tint_unittests_diagnostic_src") {
sources = [
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/diagnostic/formatter_test.cc",
"../src/diagnostic/printer_test.cc",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
]
}
tint_unittests_source_set("tint_unittests_inspector_src") {
sources = [
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/inspector/inspector_test.cc",
Factor out the InspectorHelper from the Inspector test file This class is very large, so factoring it out into its own seperate file. Also renaming it to InspectorBuilder, in anticipation of adding tests that don't build their state through explicitly calling into the ProgramBuilder. BUG=tint:967 Change-Id: I316458a969479200edf6962ab2008180f9a1e7e2 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/57800 Auto-Submit: Ryan Harrison <rharrison@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-07-13 15:34:45 +00:00
"../src/inspector/test_inspector_builder.cc",
"../src/inspector/test_inspector_builder.h",
Handle complex cases in Inspector::GenerateSamplerTargets This code was implicitly assuming that all resources it was looking for would be directly referenced at the intrinsic callsite, and not passed via function parameters. This was causing a crash in more complex cases. The inspector code has been updated to handle cases where the resources are not being directly referenced. Unneeded calls to GenerateSamplerTargets() are removed. Utility function GetOriginatingResources() is added to handle walking up call sites to resolve resources. Text shader based test runner is added to the Inspector tests to make expressing complex tests easier. BUG=tint:967 Change-Id: I2ecb6d57c518003da59f38b261bae4d62ce7e6ac Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/59340 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Auto-Submit: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-07-27 15:42:51 +00:00
"../src/inspector/test_inspector_runner.cc",
"../src/inspector/test_inspector_runner.h",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
]
}
tint_unittests_source_set("tint_unittests_resolver_src") {
sources = [
Move array accessor tests to their own file No new tests or any changes, just moving to a separate file in preparation for adding lots more of these tests soon. Change-Id: Iaa7eef52384e702c395a6db312fef19e22507644 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/63580 Kokoro: Kokoro <noreply+kokoro@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: James Price <jrprice@google.com>
2021-09-07 14:48:24 +00:00
"../src/resolver/array_accessor_test.cc",
Resolver: Check that initializers and assignments are valid This performs very basic type verification for assignments and variable initializers. Pulls part of the validation logic out of the Validator into the Resolver. Involves fixing up a bunch of broken tests. Bug: tint:642 Fixed: tint:631 Change-Id: Ifbdc139ff7eeab810856e0ba9e3c380c6555ec20 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45281 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: David Neto <dneto@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-03-18 21:14:44 +00:00
"../src/resolver/assignment_validation_test.cc",
resolver: Resolve atomic types Bug: tint:892 Change-Id: Ib595378b3b126a8f6bc4712ece943ba70816cb46 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/54647 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-06-17 19:56:14 +00:00
"../src/resolver/atomics_test.cc",
"../src/resolver/atomics_validation_test.cc",
resolver: Fixes for bitcasts Fix dependency graph traversal for bitcasts. These were not being traversed, leading to an ICE if the bitcast type was an alias, as the symbol was not resolved for later use by the resolver. Add missing validation for bitcasts. We were permitting any bitcast that wasn't a being cast to a pointer type, when the spec only allows: * numeric_scalar to numeric_scalar * vecN<numeric_scalar> to vecN<numeric_scalar> Add lots of tests. Fixed: chromium:1276320 Change-Id: I9e5487ec7649ac543f73fc878e7e282bf932d8cb Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/71681 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-12-03 21:29:13 +00:00
"../src/resolver/bitcast_validation_test.cc",
Finish moving call validation from Validator to Resolver Call validation was already implemented in Resolver. This change completes it by deleting the relevant code in Validator, and moving and updating the builtins validation test to use the Resolver. Also added the "v-0004" error code for when detecting recursion, as was done for the similar error in the Validator. Bug: tint:642 Bug: tint:487 Change-Id: If7973bfd2d19681a0cbf48c6d427e17a3b927cde Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45463 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-03-22 17:42:06 +00:00
"../src/resolver/builtins_validation_test.cc",
Validate function call arguments - Add resolver/call_test.cc for new unit tests, and move a couple that were in resolver/validation_test.cc to it - Fix CalculateArrayLength transform so that it passes the address of the u32 it creates to the internal function - Fix tests broken as a result of this change Bug: tint:664 Change-Id: If713f9828790cd51224d2392d42c01c0057cb652 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53920 Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-09 20:17:59 +00:00
"../src/resolver/call_test.cc",
resolver: Validate that a call statement has no return value Also split out validation tests from call_test.cc into call_validation_test.cc. Bug: tint:886 Change-Id: I1e1dee9b7c348363e89080cdecd3119cc004658f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/54063 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-06-11 13:22:27 +00:00
"../src/resolver/call_validation_test.cc",
sem: Add CompoundStatement This change introduces sem::CompoundStatement, a new base class for statements that can hold other statements. sem::BlockStatements now derives from sem::CompoundStatement, and this change introduces the following new CompoundStatements: * `sem::IfStatement` * `sem::ElseStatement` * `sem::ForLoopStatement` * `sem::LoopStatement` * `sem::SwitchStatement`. These new CompoundStatements are now inserted into the semantic tree as now documented in `docs/compound_statements.md`. The `sem::BlockStatement::FindFirstParent()` methods have been moved down to `sem::Statement`. The `Resolver::BlockScope()` method has been replaced with `Resolver::Scope()` which now maintains the `current_statement_`, `current_compound_statement_ ` and `current_block_`. This simplifies statement nesting. The most significant change in behavior is that statements now always have a parent, so calling Block() on the initializer or continuing of a for-loop statement will now return the BlockStatement that holds the for-loop. Before this would return nullptr. Fixed: tint:979 Change-Id: I90e38fd719da2a281ed9210e975ab96171cb6842 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/57707 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-07-14 09:44:41 +00:00
"../src/resolver/compound_statement_test.cc",
Move Switch validation from Validator to Resolver * Formerly, we reported the same error message if we detected no default clause or more than one. I made it so that we output a different error message for each. This makes it more clear, and in the case of more than one, the error source location points at the second default clause, rather than at the switch statement. * Add functions to ProgramBuilder to more easily define switch and case statements. * Fix broken tests as a result of this change. Bug: tint:642 Change-Id: Iab4e610a563165862d9bc190772d32a4dd24ac45 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45880 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-03-25 12:55:27 +00:00
"../src/resolver/control_block_validation_test.cc",
Move struct validation from Validator to Resolver * Moved Validator::ValidateConstructedType, which only validated structs, to Resolver as ValidateStructure. * Moved relevant tests to new files, and also updated all failing tests to validate Source location. * Fixed other tests that broke now that we're validating structs. Bug: tint:642 Change-Id: Iefc08ef548f52d8c3798d814d2183c56d1236c2d Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45160 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-03-18 17:59:54 +00:00
"../src/resolver/decoration_validation_test.cc",
resolver: Add dependency-graph analysis Performs a module-scope (global) declaration dependency analysis, so that out-of-order global declarations can be re-ordered into dependency order for consumption by the resolver. The WGSL working group are currently debating whether out-of-order declarations should be included in WebGPU V1, so this implementation currently errors if module-scope declarations are declared out-of-order, and the resolver does not currently use this sorted global list. The analysis does however provide significantly better error diagnostics when cyclic dependencies are formed, and when globals are declared out-of-order. The DependencyGraph also correctly now detects symbol collisions between functions and types (tint:1308). With this change, validation is duplicated between the DependencyGraph and the Resolver. The now-unreachable validation will be removed from the Resolver with a followup change. Fixed: tint:1308 Bug: tint:1266 Change-Id: I809c23a069a86cf429f5ec8ef3ad9a98246766ab Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69381 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@chromium.org> Reviewed-by: David Neto <dneto@google.com>
2021-11-22 11:44:57 +00:00
"../src/resolver/dependency_graph_test.cc",
[resolver] Add entry point IO validation Checks the following things: - Non-struct entry point parameters must have pipeline IO attributes - Non-struct entry point return type must have a pipeline IO attribute - Structs used as entry point parameters and return values much have pipeline IO attributes on every member - Structs used as entry point parameters and return values cannot have runtime array or nested struct members - Multiple pipeline IO attributes on a parameter, return type, or struct member is not allowed - Any given builtin and location attribute can only appear once for the return type, and across all parameters Removed tests for nested structs from the SPIR-V transform/backend. Fixed a couple of other tests with missing pipeline IO attributes. Fixed: tint:512 Change-Id: I4c48fe24099333c8c7fcd45934c09baa6830883c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/46701 Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-06 15:51:47 +00:00
"../src/resolver/entry_point_validation_test.cc",
Move function validation from Validator to Resolver * Fixed many tests that now failed validation. Most of the time, functions declared that they returned a type, but with no return statement. * ProgramBuilder::WrapInFunction now returns the function is creates, and std::moves its StatementList. * ProgramBuilder: Added Return function to create ast::ReturnStatements more easily. Bug: tint:642 Change-Id: I3011314e66e264ebd7b89bf9271392391be6a0e5 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45382 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-03-19 14:04:51 +00:00
"../src/resolver/function_validation_test.cc",
s/sharable/shareable Fixed: tint:660 Change-Id: I5b98597acb771f6b07a16a85d85d8ae5249e495d Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45283 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-03-18 21:03:24 +00:00
"../src/resolver/host_shareable_validation_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/resolver/intrinsic_test.cc",
resolver: Validate pipline stage use for intrinsics Use the new [[stage()]] decorations in intrinsics.def to validate that intrinsics are only called from the correct pipeline stages. Fixed: tint:657 Change-Id: I9efda26369c45c6f816bdaa53408d3909db403a1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53084 Reviewed-by: Ryan Harrison <rharrison@chromium.org> Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-06-03 16:07:34 +00:00
"../src/resolver/intrinsic_validation_test.cc",
s/sharable/shareable Fixed: tint:660 Change-Id: I5b98597acb771f6b07a16a85d85d8ae5249e495d Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45283 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-03-18 21:03:24 +00:00
"../src/resolver/is_host_shareable_test.cc",
Implement Default Struct Layout Implements https://github.com/gpuweb/gpuweb/pull/1447 SPIR-V Reader is still TODO, but continues to function as the offset decoration is still supported. Bug: tint:626 Bug: tint:629 Change-Id: Id574eb3a5c6729559382812de37b23f0c68fd406 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/43640 Commit-Queue: Ben Clayton <bclayton@chromium.org> Reviewed-by: David Neto <dneto@google.com>
2021-03-15 10:43:11 +00:00
"../src/resolver/is_storeable_test.cc",
wsgl-writer: emit inf, nan, subnormal as hex float Signed zeros are emitted. Subormal numbers are emitted as hex float. Handling Inf and NaN is unresolved in the spec. See https://github.com/gpuweb/gpuweb/issues/1769 NaN tests are disabled, due to platform dependence. Windows x86-64 seems to always set the high mantissa bit on NaNs. Fixed: tint:76 Change-Id: I06c69b93b04c869a63ec2f574022996acc7a6dbe Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/52321 Commit-Queue: David Neto <dneto@google.com> Auto-Submit: David Neto <dneto@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-05-27 20:46:46 +00:00
"../src/resolver/pipeline_overridable_constant_test.cc",
Implement Pointers and References This change implements pointers and references as described by the WGSL specification change in https://github.com/gpuweb/gpuweb/pull/1569. reader/spirv: * Now emits address-of `&expr` and indirection `*expr` operators as needed. * As an identifier may now resolve to a pointer or reference type depending on whether the declaration is a `var`, `let` or parameter, `Function::identifier_values_` has been changed from an ID set to an ID -> Type* map. resolver: * Now correctly resolves all expressions to either a value type, reference type or pointer type. * Validates pointer / reference rules on assignment, `var` and `let` construction, and usage. * Handles the address-of and indirection operators. * No longer does any implicit loads of pointer types. * Storage class validation is still TODO (crbug.com/tint/809) writer/spirv: * Correctly handles variables and expressions of pointer and reference types, emitting OpLoads where necessary. test: * Lots of new test cases Fixed: tint:727 Change-Id: I77d3281590e35e5a3122f5b74cdeb71a6fe51f74 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/50740 Commit-Queue: Ben Clayton <bclayton@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-05-18 10:28:48 +00:00
"../src/resolver/ptr_ref_test.cc",
"../src/resolver/ptr_ref_validation_test.cc",
resolver: Implement Behavior Analysis This change implements the behavior analysis for expressions and statements as described in: https://www.w3.org/TR/WGSL/#behaviors This CL makes no changes to the validation rules. This will be done as a followup change. Bug: tint:1302 Change-Id: If0a251a7982ea15ff5d93b54a5cc5ed03ba60608 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/68408 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-12-03 15:49:34 +00:00
"../src/resolver/resolver_behavior_test.cc",
[resolver]: Begin constant value evaluation Move the bulk of the constant evaulation logic out of transform::FoldConstants and into Resolver and sem::Expression. transform::FoldConstants now replace TypeConstructor nodes that have a constant value on the expression. This is ground work to: * Cleaning up the HLSL uniform buffer indexing, which is `/` and `%` arithmatic heavy * Prepares us to handle `constexpr` when it lands in the spec * Provide a centralized place to do constant evaluation, instead of the having similar logic scattered around the codebase. Change-Id: I3e2f542be692046a8d243b62a82556db519953e7 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/57426 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-07-13 12:18:13 +00:00
"../src/resolver/resolver_constants_test.cc",
Fix gn check in Chromium - Move headers from libtint_sem_src into libtint_core_all_src so that there is no circular header dependency between headers of these two targets. libtint_sem_src is now just a hack to have a different name for a few .cc files. - Made libtint_core_src publicly depend on its deps so that headers of libtint_core_all_src are made visible through it. - Added spvtools dependencies in a couple places that were mistakenly removed in a previous commit. - Moved helpers common to multiple unittest targets out of tint_unittests_core_src and into a tint_test_helpers target that all tint_unittests_source_set depend on. - Ran GN format that reordered some lists a bit. Bug: None Change-Id: I544a73d73366be9dd2ac8e56c7593fd9f2b86cf8 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48600 Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-21 09:18:52 +00:00
"../src/resolver/resolver_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/resolver/resolver_test_helper.cc",
"../src/resolver/resolver_test_helper.h",
Reland "Validate storage class constraints" This is a reland of fd5829e5ead28012956b214940392407aa455530 Original change's description: > Validate storage class constraints > > As defined by https://gpuweb.github.io/gpuweb/wgsl/#storage-class-layout-constraints > > Bug: tint:643 > Change-Id: I9c78ba69a792a80c263a17b0a6e9b4810fdb7f30 > Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56780 > Kokoro: Kokoro <noreply+kokoro@google.com> > Commit-Queue: Antonio Maiorano <amaiorano@google.com> > Reviewed-by: Ben Clayton <bclayton@google.com> Bug: tint:643 Change-Id: I62036d615a062597339a9d130b7ccf49b5be26c7 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/58120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-07-15 19:09:25 +00:00
"../src/resolver/storage_class_layout_validation_test.cc",
Resolver: Validate <storage> var types https://gpuweb.github.io/gpuweb/wgsl/#variable-declaration Variables in the storage storage class and variables with a storage texture type must have an access attribute applied to the store type. https://gpuweb.github.io/gpuweb/wgsl/#module-scope-variables A variable in the storage storage class is a storage buffer variable. Its store type must be a host-shareable structure type with block attribute, satisfying the storage class constraints. Fixup tests, including those that were producing warnings about `var <in>` The WGSL writer seems to want to put a newline after every decoration block, leading to some ugly output. I'll fix this as a separate change. Fixes: tint:531 Fixes: tint:692 Change-Id: If09d987477247ab4a7c635f6ee6e616a06061515 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/47763 Commit-Queue: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com> Reviewed-by: Ryan Harrison <rharrison@chromium.org>
2021-04-16 09:26:14 +00:00
"../src/resolver/storage_class_validation_test.cc",
Implement Default Struct Layout Implements https://github.com/gpuweb/gpuweb/pull/1447 SPIR-V Reader is still TODO, but continues to function as the offset decoration is still supported. Bug: tint:626 Bug: tint:629 Change-Id: Id574eb3a5c6729559382812de37b23f0c68fd406 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/43640 Commit-Queue: Ben Clayton <bclayton@chromium.org> Reviewed-by: David Neto <dneto@google.com>
2021-03-15 10:43:11 +00:00
"../src/resolver/struct_layout_test.cc",
[resolver] Track pipeline stage usages for structs This will be used by the generators to determine how to handle location decorations. Change-Id: Ib0e0ce852a5da3819781b402c5625a440c4c9544 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/46400 Auto-Submit: James Price <jrprice@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-03-31 15:42:17 +00:00
"../src/resolver/struct_pipeline_stage_use_test.cc",
Resolver: Track storage class usages of structures This will be used to validate layout rules, as well as preventing illegal types from being used in a uniform / storage buffer. Also: Cleanup logic around VariableDeclStatement This was spread across 3 places, entirely unnecessarily. Bug: tint:643 Change-Id: I9d309c3a5dfb5676984f49ce51763a97bcac93bb Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45125 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: James Price <jrprice@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-03-17 22:47:33 +00:00
"../src/resolver/struct_storage_class_use_test.cc",
Add type inference support to Resolver There is still no way to spell this out in WGSL, but this adds support for VariableDecls with an ast::Variable that has nullptr type. In this case, the Resolver uses the type of the rhs (constructor expression), which is stored in semantic::Variable. Added tests for resolving inferred types from constructor, arithmetic, and call expressions. Bug: tint:672 Change-Id: I3dcfd18adecebc8b969373d2ac72c21891c21a87 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/46160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-03-31 12:46:52 +00:00
"../src/resolver/type_constructor_validation_test.cc",
Move struct validation from Validator to Resolver * Moved Validator::ValidateConstructedType, which only validated structs, to Resolver as ValidateStructure. * Moved relevant tests to new files, and also updated all failing tests to validate Source location. * Fixed other tests that broke now that we're validating structs. Bug: tint:642 Change-Id: Iefc08ef548f52d8c3798d814d2183c56d1236c2d Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45160 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-03-18 17:59:54 +00:00
"../src/resolver/type_validation_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/resolver/validation_test.cc",
Implement Pointers and References This change implements pointers and references as described by the WGSL specification change in https://github.com/gpuweb/gpuweb/pull/1569. reader/spirv: * Now emits address-of `&expr` and indirection `*expr` operators as needed. * As an identifier may now resolve to a pointer or reference type depending on whether the declaration is a `var`, `let` or parameter, `Function::identifier_values_` has been changed from an ID set to an ID -> Type* map. resolver: * Now correctly resolves all expressions to either a value type, reference type or pointer type. * Validates pointer / reference rules on assignment, `var` and `let` construction, and usage. * Handles the address-of and indirection operators. * No longer does any implicit loads of pointer types. * Storage class validation is still TODO (crbug.com/tint/809) writer/spirv: * Correctly handles variables and expressions of pointer and reference types, emitting OpLoads where necessary. test: * Lots of new test cases Fixed: tint:727 Change-Id: I77d3281590e35e5a3122f5b74cdeb71a6fe51f74 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/50740 Commit-Queue: Ben Clayton <bclayton@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-05-18 10:28:48 +00:00
"../src/resolver/var_let_test.cc",
"../src/resolver/var_let_validation_test.cc",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
]
Add tests for fuzzers::RandomGenerator BUG=tint:1019 Change-Id: Ia462080877a97348c5589bfa71231a832a7ebfd3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70081 Auto-Submit: Ryan Harrison <rharrison@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-24 14:55:05 +00:00
deps = [ ":tint_unittests_ast_src" ]
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
}
tint_unittests_source_set("tint_unittests_sem_src") {
sources = [
sem: Add atomic type Bug: tint:892 Change-Id: I4826b71e5911c7f66019a956685675e9504551cf Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/54646 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-06-17 15:48:39 +00:00
"../src/sem/atomic_type_test.cc",
Move type/* files to sem/ directory Bug: tint:724 Change-Id: I45d83d32dbce1fbee265810615c18219ef879b20 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48363 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-19 22:54:43 +00:00
"../src/sem/bool_type_test.cc",
Implement texture_depth_multisampled_2d Implemented for all readers and writers. Cleaned up some verbose code in sem::Function and the Inspector in the process. Fixed: tint:1032 Change-Id: Ia6f2f59e6d2e511c89160b97be990e8b7c9828d9 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/59664 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com> Reviewed-by: James Price <jrprice@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: Ben Clayton <bclayton@google.com>
2021-07-26 22:19:48 +00:00
"../src/sem/depth_multisampled_texture_type_test.cc",
Move type/* files to sem/ directory Bug: tint:724 Change-Id: I45d83d32dbce1fbee265810615c18219ef879b20 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48363 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-19 22:54:43 +00:00
"../src/sem/depth_texture_type_test.cc",
"../src/sem/external_texture_type_test.cc",
"../src/sem/f32_type_test.cc",
"../src/sem/i32_type_test.cc",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
"../src/sem/intrinsic_test.cc",
Move type/* files to sem/ directory Bug: tint:724 Change-Id: I45d83d32dbce1fbee265810615c18219ef879b20 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48363 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-19 22:54:43 +00:00
"../src/sem/matrix_type_test.cc",
"../src/sem/multisampled_texture_type_test.cc",
"../src/sem/pointer_type_test.cc",
Add sem::Reference Bug: tint:727 Change-Id: I904d33b6c0678f478b4687fcc08589dc3cfd379d Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/50541 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-05-10 19:45:46 +00:00
"../src/sem/reference_type_test.cc",
Move type/* files to sem/ directory Bug: tint:724 Change-Id: I45d83d32dbce1fbee265810615c18219ef879b20 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48363 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-19 22:54:43 +00:00
"../src/sem/sampled_texture_type_test.cc",
"../src/sem/sampler_type_test.cc",
sem: Fold together sem::Array and sem::ArrayType There's now no need to have both. Removes a whole bunch of Sem().Get() smell, and simplifies the resolver. Also fixes a long-standing issue where an array with an explicit, but equal-to-implicit-stride attribute would result in a different type to an array without the decoration. Bug: tint:724 Fixed: tint:782 Change-Id: I0202459009cd45be427cdb621993a5a3b07ff51e Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/50301 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-05-07 20:58:34 +00:00
"../src/sem/sem_array_test.cc",
sem: Fold together sem::Struct and sem::StructType There's now no need to have both. Removes a whole bunch of Sem().Get() smell, and simplifies the resolver. Bug: tint:724 Fixed: tint:761 Change-Id: I756a32680ac52441fd6eebf6fc53dd507ef5e538 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49961 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-05-07 14:49:34 +00:00
"../src/sem/sem_struct_test.cc",
Move type/* files to sem/ directory Bug: tint:724 Change-Id: I45d83d32dbce1fbee265810615c18219ef879b20 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48363 Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-19 22:54:43 +00:00
"../src/sem/storage_texture_type_test.cc",
"../src/sem/texture_type_test.cc",
"../src/sem/type_manager_test.cc",
"../src/sem/u32_type_test.cc",
"../src/sem/vector_type_test.cc",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
]
}
tint_unittests_source_set("tint_unittests_transform_src") {
sources = [
transform: Add transform to add empty entry point Use this from the HLSL and SPIR-V sanitizers, instead of duplicating this logic for them. This is step towards removing the sanitizers completely. Change-Id: Ifa9f23d84fd3505d30a928c260181a699c5f1783 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/63582 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-09-07 18:59:21 +00:00
"../src/transform/add_empty_entry_point_test.cc",
spirv: Add block decorations with a transform Any struct which is used as the store type of a buffer variable needs to have a block decoration. If that struct is nested inside an array or another struct, we wrap it inside another struct first. This removes the SPIR-V backend's reliance on the [[block]] attribute, which will soon be deprecated and removed. Bug: tint:1324 Change-Id: Ib6ad54f24a3e4a090da9faeed699f266abcb66ba Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/72082 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-12-09 15:45:03 +00:00
"../src/transform/add_spirv_block_decoration_test.cc",
msl: Add ArrayLengthFromUniform transform Generate a uniform buffer that will receive the lengths of all storage buffers, and use this to implement calls to arrayLength(). The transform is provided with a set of mappings from storage buffer binding points to the corresponding index into the array of buffer lengths. The transform reports whether it generated the uniform buffers or not. Use this transform from the MSL sanitizer, using the binding number as the index into the array. This matches the behavior of spirv-cross, and so works with how Dawn already produces this uniform buffer. Bug: tint:256 Change-Id: I2682d2d024e8daa30f78270b8cfb6bbb32632133 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/54480 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-19 00:33:35 +00:00
"../src/transform/array_length_from_uniform_test.cc",
Add transform::BindingRemapper A transform to replace binding points and access control flags. Required by Dawn for the HLSL and MSL backends Fixed: tint:104 Fixed: tint:621 Fixed: tint:671 Change-Id: Ic6ccc4a8a7724697cc4af0fad25394a1973eeb22 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/46262 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: James Price <jrprice@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-03-29 21:25:45 +00:00
"../src/transform/binding_remapper_test.cc",
Add transform::CalculateArrayLength Used to used to replace calls to arrayLength() with a value calculated from the size of the storage buffer. Bug: tint:185 Change-Id: If7ddc8dad2ed3d20c1d76b5f48bfd2c7634f96e2 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/46877 Commit-Queue: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-04-09 10:28:48 +00:00
"../src/transform/calculate_array_length_test.cc",
Add a transform for canonicalizing entry point interfaces After the transform, an entry point's parameters will be aggregated into a single struct, and its return type will either be a struct or void. All structs in the module that have entry point IO decorations will have exactly one pipeline stage usage. This will be used to sanitize entry points for the MSL and HLSL generators. Change-Id: I7d2fcfba961d2e20326086964207747d573b6e05 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/46500 Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-03-31 17:44:27 +00:00
"../src/transform/canonicalize_entry_point_io_test.cc",
writer/hlsl: Emit UBO as an array of vector Instead of a ConstantBuffer. HLSL requires that each structure field in a UBO is 16 byte aligned. WGSL has much looser constraints with its UBO field alignment rules. Instead generate an array of uint4 vectors, and index into this, much like we index into [RW]ByteAddressBuffers for SSBOs. Extend the DecomposeStorageAccess transform to support uniforms too. This has been renamed to DecomposeMemoryAccess. Change-Id: I3868ff80af1ab3b3dddfbf5b969724cb87ef0744 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55246 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-06-18 21:15:25 +00:00
"../src/transform/decompose_memory_access_test.cc",
reader/spirv: Handle the MatrixStride decoration Add `transform::DecomposeStridedMatrix`, which replaces matrix members of storage or uniform buffer structures, that have a [[stride]] decoration, into an array of N column vectors. This is required to correctly handle `mat2x2` matrices in UBOs, as std140 rules will expect a default stride of 16 bytes, when in WGSL the default structure layout expects a stride of 8 bytes. Bug: tint:1047 Change-Id: If5ca3c6ec087bbc1ac31a8d9a657b99bf34042a4 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/59840 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-07-27 08:17:29 +00:00
"../src/transform/decompose_strided_matrix_test.cc",
Add Transform For External Textures Adds a transform to reclassify single-plane texture_external types into 2d sampled textures. Adds a unit test for the transform. Bug: dawn:728 Change-Id: Id1f88565aeacbfea47b140181c78ad122edbdae8 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49641 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-05-05 16:38:12 +00:00
"../src/transform/external_texture_transform_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/transform/first_index_offset_test.cc",
transform: Add FoldTrivialSingleUseLets This transform is intended to clean up the output of the SPIR-V reader, so that we can pattern match loops that can be transformed into a for-loop. Bug: tint:952 Change-Id: Iba58e4e1f6e20daaf7715e493df53346cdb7c89f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56766 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com> Commit-Queue: David Neto <dneto@google.com> Auto-Submit: Ben Clayton <bclayton@google.com>
2021-07-02 21:15:44 +00:00
"../src/transform/fold_constants_test.cc",
"../src/transform/fold_trivial_single_use_lets_test.cc",
[transform] Add ForLoopToLoop Transforms a for-loop into a loop. Will be required by the SPIR-V writer. Bug: tint:952 Change-Id: Iba859bd144d702cee85374f2cfcb94cd7465ca98 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/57202 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-07-08 10:00:17 +00:00
"../src/transform/for_loop_to_loop_test.cc",
HLSL: work around FXC failures when dynamically indexing arrays in structs FXC fails to compile code that assigns to dynamically-indexed fixed-size arrays in structs on internal shader variables with: error X3500: array reference cannot be used as an l-value; not natively addressable This CL detects this case, and transforms such assignments into copying out the array to a local variable, assigning to that local, and then copying the array back. Also manually regenerate SKIPs for HLSL/FXC after this change, which fixes 30 tests. Also exposes some "compilation aborted unexpectedly" now that "array reference cannot be used as an l-value" has been fixed. For tests that fail for both DXC and FXC, updating SKIPs to the DXC one to help distinguish actual FXC bugs from valid errors. Bug: tint:998 Bug: tint:1206 Change-Id: I09204d8d81ab27d1c257538ad702414ccc386543 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/71620 Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-12-13 15:55:11 +00:00
"../src/transform/localize_struct_array_assignment_test.cc",
transform: Add LoopToForLoop A Transform that attempts to convert WGSL `loop {}` statements into a for-loop statement. For-loops cause less broken behavior with FXC than our current loop constructs. Bug: tint:952 Change-Id: I0b7b1dbec9df4c004b0419d3feae53a195ec79bb Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56767 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-07-02 22:17:25 +00:00
"../src/transform/loop_to_for_loop_test.cc",
transform: Add ModuleScopeVarToEntryPointParam This is the HandleModuleScopeVars() part of the MSL sanitizer moved verbatim to a standalone transform. The transform code is unchanged, but some expected test outputs are different as this is now tested in isolation instead of along with the rest of the sanitizer transforms. This is step towards removing the sanitizers completely. Change-Id: I7be826e2119451fc2ce2891740cc94f978e7d5a1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/63583 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-09-07 18:59:21 +00:00
"../src/transform/module_scope_var_to_entry_point_param_test.cc",
Add MultiplanarExternalTextureTransform and Tests Implements MultiplanarExternalTextureTransform to allow transforming a texture_external binding into two texture_2d<f32> bindings and a uniform buffer binding. Transforms textureSampleLevel and textureLoad calls with a texture_external parameter into custom functions that can handle both single-plane RGBA or bi-planar YUV. Includes tests. Bug: dawn:1082 Change-Id: Icb6d8b0f3773feca01c833171f07230c3531f3aa Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/68620 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-11-17 12:10:16 +00:00
"../src/transform/multiplanar_external_texture_test.cc",
transform: Add NumWorkgroupsFromUniform transform This transform scans entry points for struct parameters that contain the num_workgroups builtin, and replace accesses to these members with a value loaded from a uniform buffer. This will be used by the HLSL backend to implement the num_workgroups builtin. Bug: tint:752 Change-Id: Iefab3b14af8a08a6135348fded368a06d932e915 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/63961 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-09-13 17:11:58 +00:00
"../src/transform/num_workgroups_from_uniform_test.cc",
Add transform/PadArrayElements Replaces arrays with an explicit stride with an array to a structure holding the element padded with a `[[size]]` decoration. Note that the HLSL writer is still not correctly emitting structure fields with a `[[size]]`, which will be fixed in a follow up change. Bug: tint:182 Bug: tint:895 Fixed: tint:180 Fixed: tint:649 Change-Id: Ic135dfc89309ac805507e9f39392577c7f82d154 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/54582 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-06-16 09:50:11 +00:00
"../src/transform/pad_array_elements_test.cc",
writer/msl: Hoist array and struct initializers Pull the HLSL transformation out to a standalone transform that can be used by both HLSL and MSL. The new E2E tests do not yet pass for MSL because they produce array assignments, which will be addressed in the next patch. Fixed: tint:826 Change-Id: Idc27c81ad45e3d4ab96d82663927d2fc1384618e Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/52842 Auto-Submit: James Price <jrprice@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-01 12:08:20 +00:00
"../src/transform/promote_initializers_to_const_var_test.cc",
Address late review comments from 67064 Use spec-spelling of 'constructible'. Add missing test file to 'test/BUILD.gn' See https://dawn-review.googlesource.com/c/tint/+/67064 Change-Id: Ie39773617fd0a363d63cc6449bf3905c9eb6786d Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/67380 Auto-Submit: Ben Clayton <bclayton@google.com> Commit-Queue: James Price <jrprice@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-10-25 14:26:55 +00:00
"../src/transform/remove_phonies_test.cc",
resolver: Migrate validation to behavior analysis Migrate some of the validation logic over to use the results of behavior analysis. The most significant changes are: * Unreachable-statements now consider merge-points of control flow. For example, if all branches of a if-statement or switch-statement either return or discard, the next statement will be considered unreachable. * Unreachable statements are no longer an error, but a warning. See https://github.com/gpuweb/gpuweb/issues/2378. * Statements that follow a loops that does not break, or have a conditional will now be considered unreachable. * Unreachable statements produced by the SPIR-V reader are now removed using the new RemoveUnreachableStatements transform. Some other new changes include additional validation for the continuing block for for-loops, to match the rules of a loop continuing block. The new cases this validation is testing for are not expressible in WGSL, but some transforms may produce complex continuing statements that might violate these rules. All the writers are able to decay these complex for-loop continuing statements to regular loops. Bug: tint:1302 Change-Id: I0d8a48c73d5d5c30a1cddf92cc3383a692a58e61 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/71500 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com>
2021-12-03 17:51:48 +00:00
"../src/transform/remove_unreachable_statements_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/transform/renamer_test.cc",
transform: Rename BoundArrayAccessors to Robustness We will want this transform to do more bounds and argument sanitization. Bug: tint:748 Change-Id: I38cb9623622e9f5ab85d8cd420d669ca6be77099 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56543 Auto-Submit: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-06-30 13:23:36 +00:00
"../src/transform/robustness_test.cc",
transform: Combine InlinePointerLets and Simplify They are always used together, and can actually be simplified when combined. This transform cannot currently deal with shadowing, and will need need to depend on a new 'unshadow' transform. Fixes a long-standing bug where we'd get an ICE if we attempt to inline a pointer let declaration with side-effects in a for-loop initializer. Fixed: tint:1321 Bug: tint:819 Change-Id: I236fed688e33a4996e47310b5ece44c991b5249f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70661 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-11-23 20:45:51 +00:00
"../src/transform/simplify_pointers_test.cc",
Add transform to strip entry points from a module Remove the Generator::GenerateEntryPoint() APIs as they were mostly unimplemented and not used by anything except the Tint sample app, which now uses the new transform. Change-Id: I1ccb303d6c3aa15e622c193d33b753e22bf39a95 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: James Price <jrprice@google.com>
2021-04-28 15:33:03 +00:00
"../src/transform/single_entry_point_test.cc",
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
"../src/transform/test_helper.h",
Add Transform::CreateASTTypeFor() Reconstructs the AST nodes needed to build the given semantic type. Bug: tint:724 Change-Id: Iadf97a47b68088a6a1eb1e6871fb3a7248676417 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49745 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-05-04 18:13:21 +00:00
"../src/transform/transform_test.cc",
resolver: Support shadowing Add transform::Unshadow to renamed shadowed symbols. Required by a number of other transforms. Replace Resolver symbol resolution with dep-graph. The dependency graph now performs full symbol resolution before the regular resolver pass. Make use of this instead of duplicating the effort. Simplfies code, and actually performs variable shadowing consistently. Fixed: tint:819 Bug: tint:1266 Change-Id: I595d1812aebe1d79d2d32e724ff90de36e74cf4b Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70523 Reviewed-by: David Neto <dneto@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-11-23 20:45:51 +00:00
"../src/transform/unshadow_test.cc",
Add support for matrix construction from scalars Use a transform to convert these to the vector form for the MSL and SPIR-V backends. MSL only has the scalar form from version 2.0 onwards. Fixed: tint:1123 Change-Id: I384abd9872d9eae52a10a37cbd6aa96004692e9c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/67360 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-10-25 19:20:31 +00:00
"../src/transform/vectorize_scalar_matrix_constructors_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/transform/vertex_pulling_test.cc",
Add transform/WrapArraysInStructs And replace the MSL writer's logic to do this with the transform. We need to do the same thing in HLSL, and in the future GLSL too. Partially reverts fbfde720 Change-Id: Ie280e011bc3ded8e15ccacc0aeb12da3c2407389 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/54242 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-06-16 09:19:36 +00:00
"../src/transform/wrap_arrays_in_structs_test.cc",
transform: Add ZeroInitWorkgroupMemory Zero initializes all referenced workgroup storage classed variables used by each entry point. Bug: tint:280 Fixed: tint:911 Change-Id: I3fca26a10f015f08fedef404720bbe6fd7b343a9 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55243 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-06-18 22:44:31 +00:00
"../src/transform/zero_init_workgroup_memory_test.cc",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
]
}
tint_unittests_source_set("tint_unittests_utils_src") {
sources = [
[utils]: Add TINT_DEFER() Runs the statement(s) at the end of the lexical scope Change-Id: I74de02b7204b56016c7bbe71fee4ece498d3570d Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51923 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-07-08 10:00:17 +00:00
"../src/utils/defer_test.cc",
[utils]: Add EnumSet Simple container for sets of enum values. Change-Id: I9efe5fc15e9f179cb63b64f3783235a92013a2b3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53083 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: David Neto <dneto@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-06-03 15:53:34 +00:00
"../src/utils/enum_set_test.cc",
Add utils/hash.h Contains hash combining functions borrowed and tweaked from Dawn Change-Id: I0bc9478bcdba0b923d0b01825c275290ffa29976 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/46268 Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-03-29 15:13:55 +00:00
"../src/utils/hash_test.cc",
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
"../src/utils/io/command_test.cc",
"../src/utils/io/tmpfile_test.cc",
utils: Add Lookup() A find or return-default utility. Rename `find_or_replace.h` to `map.h` which contains both these utilties that operate on maps. Change-Id: Iaa76ea3f5c5a1210e413c131dd0556c126ee5d0a Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70521 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-11-23 18:37:37 +00:00
"../src/utils/map_test.cc",
Add src/utils/math.h Move the RoundUp() and IsPowerOfTwo() methods from Resolver.cc to this file. Add tests Change-Id: Ib7af53dfa5e69083ec4fc2484da92a84c9468818 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44682 Auto-Submit: Ben Clayton <bclayton@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-03-15 13:37:41 +00:00
"../src/utils/math_test.cc",
Resolver: Traverse expressions without recursion This CL changes the way that the resolver traverses expressions to avoid stack overflows for deeply nested expressions. Instead of having the expression resolver methods call back into Expression(), add a TraverseExpressions() method that collects all the expression nodes with a simple DFS. This currently only changes the way that Expressions are traversed. We may need to do the same for statements. Bug: chromium:1246375 Change-Id: Ie81905da1b790b6dd1df9f1ac42e06593d397c21 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/63700 Auto-Submit: Ben Clayton <bclayton@google.com> Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-09-08 15:18:36 +00:00
"../src/utils/reverse_test.cc",
utils: Add TINT_SCOPED_ASSIGNMENT() A helper class and macro used to simplify scope-based assignment of variables. Change-Id: I02b3a05240a2c4628f813de931c40d8fba3cb07b Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51480 Auto-Submit: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-05-19 16:11:04 +00:00
"../src/utils/scoped_assignment_test.cc",
utils: Add ReplaceAll() string helper Helpful for template string substitutions in tests, to reduce string concat noise. Change-Id: I612084e2c267d93d79c92cf3ac0231b6b904f0e3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/66440 Commit-Queue: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-10-14 20:24:50 +00:00
"../src/utils/string_test.cc",
utils: Add utils::Transform An element-wise vector transformation utility function. Similar to JS/TS's map() method on arrays. Change-Id: I4baf52daa918f2e7bf5f9b4af13894fe66826f7c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69103 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@chromium.org> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-12 13:04:06 +00:00
"../src/utils/transform_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/utils/unique_vector_test.cc",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
]
}
tint_unittests_source_set("tint_unittests_writer_src") {
sources = [
AppendVector() - support pointer args Add tests. Fixed: tint:639 Change-Id: I6718a1fcf5a46aca36209bec037633a862d8e568 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45600 Reviewed-by: James Price <jrprice@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-03-22 21:05:47 +00:00
"../src/writer/append_vector_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/writer/float_to_string_test.cc",
writer/msl: Implement atomics Common logic between the HLSL, WGSL and MSL writers has been moved into the TextGenerator base class. Fixed: tint:892 Change-Id: I0f469516947fe64817ce6251e436da74e5e176e8 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56068 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@chromium.org> Reviewed-by: James Price <jrprice@google.com>
2021-06-29 11:53:15 +00:00
"../src/writer/text_generator_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
]
}
Rename sem_*.cc to *.cc This makes the .cc files match the header file name, which allows for the text editor "header flip" feature to actually work. The reason these files were named this way was because GN doesn't allow name conflicts in the same source set, despite the files being in different directories. This change splits the files into different source sets. To do so, we use GN templates, which also reduces duplication in each target definition. Bug: tint:724 Change-Id: I9a7ed3912e4b85b2b38d360805203f3488b86c4c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-04-19 15:20:03 +00:00
tint_unittests_source_set("tint_unittests_spv_reader_src") {
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
sources = [
"../src/reader/spirv/enum_converter_test.cc",
"../src/reader/spirv/fail_stream_test.cc",
"../src/reader/spirv/function_arithmetic_test.cc",
"../src/reader/spirv/function_bit_test.cc",
"../src/reader/spirv/function_call_test.cc",
"../src/reader/spirv/function_cfg_test.cc",
"../src/reader/spirv/function_composite_test.cc",
"../src/reader/spirv/function_conversion_test.cc",
"../src/reader/spirv/function_decl_test.cc",
"../src/reader/spirv/function_glsl_std_450_test.cc",
"../src/reader/spirv/function_logical_test.cc",
"../src/reader/spirv/function_memory_test.cc",
"../src/reader/spirv/function_misc_test.cc",
"../src/reader/spirv/function_var_test.cc",
"../src/reader/spirv/namer_test.cc",
Implement barrier intrinsics Fixed: tint:658 Change-Id: I28d5225f42dacb2b6b0cb51ce9f15951b31f0fc9 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45284 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com> Reviewed-by: Alan Baker <alanbaker@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-03-22 19:27:06 +00:00
"../src/reader/spirv/parser_impl_barrier_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/reader/spirv/parser_impl_convert_member_decoration_test.cc",
"../src/reader/spirv/parser_impl_convert_type_test.cc",
"../src/reader/spirv/parser_impl_function_decl_test.cc",
"../src/reader/spirv/parser_impl_get_decorations_test.cc",
"../src/reader/spirv/parser_impl_handle_test.cc",
"../src/reader/spirv/parser_impl_import_test.cc",
"../src/reader/spirv/parser_impl_module_var_test.cc",
"../src/reader/spirv/parser_impl_named_types_test.cc",
"../src/reader/spirv/parser_impl_test.cc",
Add --dump-spirv option to tint_unittests The --dump-spirv option tells tint_unittests to output the SPIR-V assembly text for a module which did not make the SPIR-V reader fail. This lets us get extract a corpus of SPIR-V modules, and lets us more easily verify that the test shaders are valid in the first place. Also: - Add test/extract-spvasm.py to split that output to separate SPIR-V assembly files - Add optional second argument test/test-all.sh to specify a directory look for input files. - BUILD.gn: Add dependency from //test:tint_unittests_main to //test:tint_unittests_config to pick up source dependency on the internal header of the SPIRV-Tools optimizer, needed by the indirection through src/reader/spirv/parser_impl_test_helper.h This is useful for bulk testing Fixed: tint:756 Change-Id: I4fe232ac736003f7d9be35544328302d652381ea Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49605 Auto-Submit: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-05-05 09:46:31 +00:00
"../src/reader/spirv/parser_impl_test_helper.cc",
"../src/reader/spirv/parser_impl_test_helper.h",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/reader/spirv/parser_impl_user_name_test.cc",
"../src/reader/spirv/parser_test.cc",
Remove sem::AccessControl In preparation for implementing https://github.com/gpuweb/gpuweb/issues/1604, this change removes the sem::AccessControl node. Instead, the ast::AccessControl::Access enum is now on the sem::StorageTexture class, as well as on sem::Variable. For sem::Variable, the field is set when the variable's type is either a storage buffer or a storage texture. Bug: tint:802 Change-Id: Id479af36b401d067b015027923f4e715f5f69f25 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51020 Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-05-14 17:51:13 +00:00
"../src/reader/spirv/parser_type_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/reader/spirv/spirv_tools_helpers_test.cc",
"../src/reader/spirv/spirv_tools_helpers_test.h",
"../src/reader/spirv/usage_test.cc",
]
Add tests for fuzzers::RandomGenerator BUG=tint:1019 Change-Id: Ia462080877a97348c5589bfa71231a832a7ebfd3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70081 Auto-Submit: Ryan Harrison <rharrison@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-24 14:55:05 +00:00
deps = [ "${tint_root_dir}/src:libtint_spv_reader_src" ]
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
}
Rename sem_*.cc to *.cc This makes the .cc files match the header file name, which allows for the text editor "header flip" feature to actually work. The reason these files were named this way was because GN doesn't allow name conflicts in the same source set, despite the files being in different directories. This change splits the files into different source sets. To do so, we use GN templates, which also reduces duplication in each target definition. Bug: tint:724 Change-Id: I9a7ed3912e4b85b2b38d360805203f3488b86c4c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-04-19 15:20:03 +00:00
tint_unittests_source_set("tint_unittests_spv_writer_src") {
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
sources = [
"../src/writer/spirv/binary_writer_test.cc",
"../src/writer/spirv/builder_accessor_expression_test.cc",
"../src/writer/spirv/builder_assign_test.cc",
"../src/writer/spirv/builder_binary_expression_test.cc",
"../src/writer/spirv/builder_bitcast_expression_test.cc",
"../src/writer/spirv/builder_block_test.cc",
"../src/writer/spirv/builder_call_test.cc",
"../src/writer/spirv/builder_constructor_expression_test.cc",
"../src/writer/spirv/builder_discard_test.cc",
"../src/writer/spirv/builder_entry_point_test.cc",
"../src/writer/spirv/builder_format_conversion_test.cc",
"../src/writer/spirv/builder_function_decoration_test.cc",
"../src/writer/spirv/builder_function_test.cc",
"../src/writer/spirv/builder_function_variable_test.cc",
"../src/writer/spirv/builder_global_variable_test.cc",
"../src/writer/spirv/builder_ident_expression_test.cc",
"../src/writer/spirv/builder_if_test.cc",
"../src/writer/spirv/builder_intrinsic_test.cc",
"../src/writer/spirv/builder_intrinsic_texture_test.cc",
"../src/writer/spirv/builder_literal_test.cc",
"../src/writer/spirv/builder_loop_test.cc",
"../src/writer/spirv/builder_return_test.cc",
"../src/writer/spirv/builder_switch_test.cc",
"../src/writer/spirv/builder_test.cc",
"../src/writer/spirv/builder_type_test.cc",
"../src/writer/spirv/builder_unary_op_expression_test.cc",
"../src/writer/spirv/instruction_test.cc",
"../src/writer/spirv/operand_test.cc",
writer/spirv: Clean up constant generation Requiring a temporary stack-allocated ast::Literal is an unpleasant requirement to generate a SPIR-V constant value. GenerateU32Literal() was also creating an invalid AST - the type was U32, yet an an ast::SintLiteral was used. Instead add Constant for holding a constant value, and use this as the map key. This also removes the last remaining use of ast::NullLiteral, which will be removed in the next change. Change-Id: Ia85732784075f153503dbef101ba95018eaa4bf5 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/45342 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-03-22 15:32:35 +00:00
"../src/writer/spirv/scalar_constant_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/writer/spirv/spv_dump.cc",
"../src/writer/spirv/spv_dump.h",
"../src/writer/spirv/test_helper.h",
]
Fix gn check in Chromium - Move headers from libtint_sem_src into libtint_core_all_src so that there is no circular header dependency between headers of these two targets. libtint_sem_src is now just a hack to have a different name for a few .cc files. - Made libtint_core_src publicly depend on its deps so that headers of libtint_core_all_src are made visible through it. - Added spvtools dependencies in a couple places that were mistakenly removed in a previous commit. - Moved helpers common to multiple unittest targets out of tint_unittests_core_src and into a tint_test_helpers target that all tint_unittests_source_set depend on. - Ran GN format that reordered some lists a bit. Bug: None Change-Id: I544a73d73366be9dd2ac8e56c7593fd9f2b86cf8 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48600 Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-21 09:18:52 +00:00
deps = [
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
":tint_unittests_ast_src",
Fix gn check in Chromium - Move headers from libtint_sem_src into libtint_core_all_src so that there is no circular header dependency between headers of these two targets. libtint_sem_src is now just a hack to have a different name for a few .cc files. - Made libtint_core_src publicly depend on its deps so that headers of libtint_core_all_src are made visible through it. - Added spvtools dependencies in a couple places that were mistakenly removed in a previous commit. - Moved helpers common to multiple unittest targets out of tint_unittests_core_src and into a tint_test_helpers target that all tint_unittests_source_set depend on. - Ran GN format that reordered some lists a bit. Bug: None Change-Id: I544a73d73366be9dd2ac8e56c7593fd9f2b86cf8 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48600 Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-04-21 09:18:52 +00:00
"${tint_root_dir}/src:libtint_spv_writer_src",
"${tint_spirv_tools_dir}/:spvtools",
]
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
}
Rename sem_*.cc to *.cc This makes the .cc files match the header file name, which allows for the text editor "header flip" feature to actually work. The reason these files were named this way was because GN doesn't allow name conflicts in the same source set, despite the files being in different directories. This change splits the files into different source sets. To do so, we use GN templates, which also reduces duplication in each target definition. Bug: tint:724 Change-Id: I9a7ed3912e4b85b2b38d360805203f3488b86c4c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-04-19 15:20:03 +00:00
tint_unittests_source_set("tint_unittests_wgsl_reader_src") {
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
sources = [
"../src/reader/wgsl/lexer_test.cc",
"../src/reader/wgsl/parser_impl_additive_expression_test.cc",
"../src/reader/wgsl/parser_impl_and_expression_test.cc",
"../src/reader/wgsl/parser_impl_argument_expression_list_test.cc",
"../src/reader/wgsl/parser_impl_assignment_stmt_test.cc",
"../src/reader/wgsl/parser_impl_body_stmt_test.cc",
"../src/reader/wgsl/parser_impl_break_stmt_test.cc",
"../src/reader/wgsl/parser_impl_bug_cases_test.cc",
"../src/reader/wgsl/parser_impl_call_stmt_test.cc",
"../src/reader/wgsl/parser_impl_case_body_test.cc",
"../src/reader/wgsl/parser_impl_const_expr_test.cc",
"../src/reader/wgsl/parser_impl_const_literal_test.cc",
"../src/reader/wgsl/parser_impl_continue_stmt_test.cc",
"../src/reader/wgsl/parser_impl_continuing_stmt_test.cc",
"../src/reader/wgsl/parser_impl_depth_texture_type_test.cc",
"../src/reader/wgsl/parser_impl_else_stmt_test.cc",
"../src/reader/wgsl/parser_impl_elseif_stmt_test.cc",
"../src/reader/wgsl/parser_impl_equality_expression_test.cc",
"../src/reader/wgsl/parser_impl_error_msg_test.cc",
"../src/reader/wgsl/parser_impl_error_resync_test.cc",
"../src/reader/wgsl/parser_impl_exclusive_or_expression_test.cc",
Add texture_external parsing and intrinsic overloads Adds texture_external to ParserImpl. Adds texture_external overloads to TextureSample, TextureLoad, and TextureDimensions to the intrisic table. Adds corresponding tests. Bug: dawn:728 Change-Id: I5e5557a10327f8c3d3044319decd748f812ecf3e Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48722 Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-04-22 22:47:03 +00:00
"../src/reader/wgsl/parser_impl_external_texture_type_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/reader/wgsl/parser_impl_for_stmt_test.cc",
"../src/reader/wgsl/parser_impl_function_decl_test.cc",
"../src/reader/wgsl/parser_impl_function_decoration_list_test.cc",
"../src/reader/wgsl/parser_impl_function_decoration_test.cc",
"../src/reader/wgsl/parser_impl_function_header_test.cc",
"../src/reader/wgsl/parser_impl_global_constant_decl_test.cc",
"../src/reader/wgsl/parser_impl_global_decl_test.cc",
"../src/reader/wgsl/parser_impl_global_variable_decl_test.cc",
"../src/reader/wgsl/parser_impl_if_stmt_test.cc",
"../src/reader/wgsl/parser_impl_image_storage_type_test.cc",
"../src/reader/wgsl/parser_impl_inclusive_or_expression_test.cc",
"../src/reader/wgsl/parser_impl_logical_and_expression_test.cc",
"../src/reader/wgsl/parser_impl_logical_or_expression_test.cc",
"../src/reader/wgsl/parser_impl_loop_stmt_test.cc",
"../src/reader/wgsl/parser_impl_multiplicative_expression_test.cc",
"../src/reader/wgsl/parser_impl_param_list_test.cc",
"../src/reader/wgsl/parser_impl_paren_rhs_stmt_test.cc",
"../src/reader/wgsl/parser_impl_pipeline_stage_test.cc",
"../src/reader/wgsl/parser_impl_primary_expression_test.cc",
"../src/reader/wgsl/parser_impl_relational_expression_test.cc",
reader/wgsl: Improve reserved keyword error messages And add `vec` and `mat` to the reserved keyword list (see https://github.com/gpuweb/gpuweb/pull/1896) Move these reserved keyword checks out of the lexer and into the parser. Generate a sensible error message. Add tests. Change-Id: I1876448201a2fd773f38f337b4e49bc61a7642f7 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56545 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com> Commit-Queue: Ben Clayton <bclayton@google.com>
2021-07-05 21:48:37 +00:00
"../src/reader/wgsl/parser_impl_reserved_keyword_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/reader/wgsl/parser_impl_sampled_texture_type_test.cc",
"../src/reader/wgsl/parser_impl_sampler_type_test.cc",
"../src/reader/wgsl/parser_impl_shift_expression_test.cc",
reader/wgsl: match spec rules for function calls Function calls should be parsed in `primary_expression`. Renames the old `postfix_expression` to `singular_expression`, with the recursive part now becoming `postfix_expression`. Fixed: tint:170 Change-Id: I1afad794880916db38a25d73447cdaf84abd6584 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/49464 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-04-29 15:02:15 +00:00
"../src/reader/wgsl/parser_impl_singular_expression_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/reader/wgsl/parser_impl_statement_test.cc",
"../src/reader/wgsl/parser_impl_statements_test.cc",
"../src/reader/wgsl/parser_impl_storage_class_test.cc",
"../src/reader/wgsl/parser_impl_storage_texture_type_test.cc",
"../src/reader/wgsl/parser_impl_struct_body_decl_test.cc",
"../src/reader/wgsl/parser_impl_struct_decl_test.cc",
"../src/reader/wgsl/parser_impl_struct_decoration_decl_test.cc",
"../src/reader/wgsl/parser_impl_struct_decoration_test.cc",
"../src/reader/wgsl/parser_impl_struct_member_decoration_decl_test.cc",
"../src/reader/wgsl/parser_impl_struct_member_decoration_test.cc",
"../src/reader/wgsl/parser_impl_struct_member_test.cc",
"../src/reader/wgsl/parser_impl_switch_body_test.cc",
"../src/reader/wgsl/parser_impl_switch_stmt_test.cc",
"../src/reader/wgsl/parser_impl_test.cc",
"../src/reader/wgsl/parser_impl_test_helper.cc",
"../src/reader/wgsl/parser_impl_test_helper.h",
"../src/reader/wgsl/parser_impl_texture_sampler_types_test.cc",
"../src/reader/wgsl/parser_impl_type_alias_test.cc",
"../src/reader/wgsl/parser_impl_type_decl_test.cc",
"../src/reader/wgsl/parser_impl_unary_expression_test.cc",
"../src/reader/wgsl/parser_impl_variable_decl_test.cc",
"../src/reader/wgsl/parser_impl_variable_decoration_list_test.cc",
"../src/reader/wgsl/parser_impl_variable_decoration_test.cc",
"../src/reader/wgsl/parser_impl_variable_ident_decl_test.cc",
wgsl: Deprecate [[access]] decorations Handle access control on var declarations instead of via [[access]] decorations. This change does the minimal work to migrate the WGSL parser over to the new syntax. Additional changes will be needed to correctly generate defaulted access qualifiers, as well as validating access usage. The [[access]] decorations are still supported by the WGSL parser, with new deprecated warnings, but not for aliases. Example: var x : [[access(x)]] alias_to_struct; Making this work is far more effort than I want to dedicate to backwards compatibility, and I do not beleive any real-world usage will be doing this. Still TODO: * Adding access control as the optional, third parameter to ptr<>. * Calculating default accesses for the various storage types. * Validating usage of variables against the different accesses. Bug: tint:846 Change-Id: If8ca82e5d16ec319ecd01f9a2cafffd930963bde Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53088 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: James Price <jrprice@google.com> Reviewed-by: David Neto <dneto@google.com> Kokoro: Kokoro <noreply+kokoro@google.com>
2021-06-04 20:41:47 +00:00
"../src/reader/wgsl/parser_impl_variable_qualifier_test.cc",
Validate function call arguments - Add resolver/call_test.cc for new unit tests, and move a couple that were in resolver/validation_test.cc to it - Fix CalculateArrayLength transform so that it passes the address of the u32 it creates to the internal function - Fix tests broken as a result of this change Bug: tint:664 Change-Id: If713f9828790cd51224d2392d42c01c0057cb652 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53920 Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-09 20:17:59 +00:00
"../src/reader/wgsl/parser_impl_variable_stmt_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/reader/wgsl/parser_test.cc",
"../src/reader/wgsl/token_test.cc",
]
Add tests for fuzzers::RandomGenerator BUG=tint:1019 Change-Id: Ia462080877a97348c5589bfa71231a832a7ebfd3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70081 Auto-Submit: Ryan Harrison <rharrison@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-24 14:55:05 +00:00
deps = [ "${tint_root_dir}/src:libtint_wgsl_reader_src" ]
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
}
Rename sem_*.cc to *.cc This makes the .cc files match the header file name, which allows for the text editor "header flip" feature to actually work. The reason these files were named this way was because GN doesn't allow name conflicts in the same source set, despite the files being in different directories. This change splits the files into different source sets. To do so, we use GN templates, which also reduces duplication in each target definition. Bug: tint:724 Change-Id: I9a7ed3912e4b85b2b38d360805203f3488b86c4c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-04-19 15:20:03 +00:00
tint_unittests_source_set("tint_unittests_wgsl_writer_src") {
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
sources = [
"../src/writer/wgsl/generator_impl_alias_type_test.cc",
"../src/writer/wgsl/generator_impl_array_accessor_test.cc",
"../src/writer/wgsl/generator_impl_assign_test.cc",
"../src/writer/wgsl/generator_impl_binary_test.cc",
"../src/writer/wgsl/generator_impl_bitcast_test.cc",
"../src/writer/wgsl/generator_impl_block_test.cc",
"../src/writer/wgsl/generator_impl_break_test.cc",
"../src/writer/wgsl/generator_impl_call_test.cc",
"../src/writer/wgsl/generator_impl_case_test.cc",
"../src/writer/wgsl/generator_impl_cast_test.cc",
"../src/writer/wgsl/generator_impl_constructor_test.cc",
"../src/writer/wgsl/generator_impl_continue_test.cc",
"../src/writer/wgsl/generator_impl_discard_test.cc",
"../src/writer/wgsl/generator_impl_fallthrough_test.cc",
"../src/writer/wgsl/generator_impl_function_test.cc",
"../src/writer/wgsl/generator_impl_global_decl_test.cc",
"../src/writer/wgsl/generator_impl_identifier_test.cc",
"../src/writer/wgsl/generator_impl_if_test.cc",
wsgl-writer: emit inf, nan, subnormal as hex float Signed zeros are emitted. Subormal numbers are emitted as hex float. Handling Inf and NaN is unresolved in the spec. See https://github.com/gpuweb/gpuweb/issues/1769 NaN tests are disabled, due to platform dependence. Windows x86-64 seems to always set the high mantissa bit on NaNs. Fixed: tint:76 Change-Id: I06c69b93b04c869a63ec2f574022996acc7a6dbe Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/52321 Commit-Queue: David Neto <dneto@google.com> Auto-Submit: David Neto <dneto@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-05-27 20:46:46 +00:00
"../src/writer/wgsl/generator_impl_literal_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/writer/wgsl/generator_impl_loop_test.cc",
"../src/writer/wgsl/generator_impl_member_accessor_test.cc",
"../src/writer/wgsl/generator_impl_return_test.cc",
"../src/writer/wgsl/generator_impl_switch_test.cc",
"../src/writer/wgsl/generator_impl_test.cc",
"../src/writer/wgsl/generator_impl_type_test.cc",
"../src/writer/wgsl/generator_impl_unary_op_test.cc",
"../src/writer/wgsl/generator_impl_variable_decl_statement_test.cc",
"../src/writer/wgsl/generator_impl_variable_test.cc",
"../src/writer/wgsl/test_helper.h",
]
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
deps = [
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
":tint_unittests_ast_src",
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
"${tint_root_dir}/src:libtint_wgsl_writer_src",
]
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
}
Rename sem_*.cc to *.cc This makes the .cc files match the header file name, which allows for the text editor "header flip" feature to actually work. The reason these files were named this way was because GN doesn't allow name conflicts in the same source set, despite the files being in different directories. This change splits the files into different source sets. To do so, we use GN templates, which also reduces duplication in each target definition. Bug: tint:724 Change-Id: I9a7ed3912e4b85b2b38d360805203f3488b86c4c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-04-19 15:20:03 +00:00
tint_unittests_source_set("tint_unittests_msl_writer_src") {
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
sources = [
"../src/writer/msl/generator_impl_array_accessor_test.cc",
"../src/writer/msl/generator_impl_assign_test.cc",
"../src/writer/msl/generator_impl_binary_test.cc",
"../src/writer/msl/generator_impl_bitcast_test.cc",
"../src/writer/msl/generator_impl_block_test.cc",
"../src/writer/msl/generator_impl_break_test.cc",
"../src/writer/msl/generator_impl_call_test.cc",
"../src/writer/msl/generator_impl_case_test.cc",
"../src/writer/msl/generator_impl_cast_test.cc",
"../src/writer/msl/generator_impl_constructor_test.cc",
"../src/writer/msl/generator_impl_continue_test.cc",
"../src/writer/msl/generator_impl_discard_test.cc",
"../src/writer/msl/generator_impl_function_test.cc",
"../src/writer/msl/generator_impl_identifier_test.cc",
"../src/writer/msl/generator_impl_if_test.cc",
"../src/writer/msl/generator_impl_import_test.cc",
"../src/writer/msl/generator_impl_intrinsic_test.cc",
"../src/writer/msl/generator_impl_intrinsic_texture_test.cc",
"../src/writer/msl/generator_impl_loop_test.cc",
"../src/writer/msl/generator_impl_member_accessor_test.cc",
"../src/writer/msl/generator_impl_module_constant_test.cc",
"../src/writer/msl/generator_impl_return_test.cc",
Add HLSL/MSL generator options for ArrayLengthFromUniform ArrayLengthFromUniform is needed for correct bounds checks on dynamic storage buffers on D3D12. The intrinsic GetDimensions does not return the actual size of the buffer binding. ArrayLengthFromUniform is updated to output the indices of the uniform buffer that are statically used. This allows Dawn to minimize the amount of data needed to upload into the uniform buffer. These output indices are returned on the HLSL/MSL generator result. ArrayLengthFromUniform is also updated to allow only some of the arrayLength calls to be replaced with uniform buffer loads. For HLSL output, the remaining arrayLength computations will continue to use GetDimensions(). For MSL, it is invalid to not specify an index into the uniform buffer for all storage buffers. After Dawn is updated to use the array_length_from_uniform option in the Metal backend, the buffer_size_ubo_index member for MSL output may be removed. Bug: dawn:429 Change-Id: I9da4ec4a20882e9f1bfa5bb026725d72529eff26 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69301 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com> Commit-Queue: Austin Eng <enga@chromium.org>
2021-11-19 04:11:33 +00:00
"../src/writer/msl/generator_impl_sanitizer_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/writer/msl/generator_impl_switch_test.cc",
"../src/writer/msl/generator_impl_test.cc",
"../src/writer/msl/generator_impl_type_test.cc",
"../src/writer/msl/generator_impl_unary_op_test.cc",
"../src/writer/msl/generator_impl_variable_decl_statement_test.cc",
"../src/writer/msl/test_helper.h",
]
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
deps = [
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
":tint_unittests_ast_src",
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
"${tint_root_dir}/src:libtint_msl_writer_src",
]
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
}
Rename sem_*.cc to *.cc This makes the .cc files match the header file name, which allows for the text editor "header flip" feature to actually work. The reason these files were named this way was because GN doesn't allow name conflicts in the same source set, despite the files being in different directories. This change splits the files into different source sets. To do so, we use GN templates, which also reduces duplication in each target definition. Bug: tint:724 Change-Id: I9a7ed3912e4b85b2b38d360805203f3488b86c4c Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/48160 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-04-19 15:20:03 +00:00
tint_unittests_source_set("tint_unittests_hlsl_writer_src") {
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
sources = [
"../src/writer/hlsl/generator_impl_array_accessor_test.cc",
"../src/writer/hlsl/generator_impl_assign_test.cc",
"../src/writer/hlsl/generator_impl_binary_test.cc",
"../src/writer/hlsl/generator_impl_bitcast_test.cc",
"../src/writer/hlsl/generator_impl_block_test.cc",
"../src/writer/hlsl/generator_impl_break_test.cc",
"../src/writer/hlsl/generator_impl_call_test.cc",
"../src/writer/hlsl/generator_impl_case_test.cc",
"../src/writer/hlsl/generator_impl_cast_test.cc",
"../src/writer/hlsl/generator_impl_constructor_test.cc",
"../src/writer/hlsl/generator_impl_continue_test.cc",
"../src/writer/hlsl/generator_impl_discard_test.cc",
"../src/writer/hlsl/generator_impl_function_test.cc",
"../src/writer/hlsl/generator_impl_identifier_test.cc",
"../src/writer/hlsl/generator_impl_if_test.cc",
"../src/writer/hlsl/generator_impl_import_test.cc",
"../src/writer/hlsl/generator_impl_intrinsic_test.cc",
"../src/writer/hlsl/generator_impl_intrinsic_texture_test.cc",
"../src/writer/hlsl/generator_impl_loop_test.cc",
"../src/writer/hlsl/generator_impl_member_accessor_test.cc",
"../src/writer/hlsl/generator_impl_module_constant_test.cc",
"../src/writer/hlsl/generator_impl_return_test.cc",
"../src/writer/hlsl/generator_impl_sanitizer_test.cc",
"../src/writer/hlsl/generator_impl_switch_test.cc",
"../src/writer/hlsl/generator_impl_test.cc",
"../src/writer/hlsl/generator_impl_type_test.cc",
"../src/writer/hlsl/generator_impl_unary_op_test.cc",
"../src/writer/hlsl/generator_impl_variable_decl_statement_test.cc",
writer/hlsl: Emit workgroup variables Fixed: tint:688 Change-Id: I3f0f3fd24734421f8d36a94c501aed9881ec10b0 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/46379 Commit-Queue: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: James Price <jrprice@google.com>
2021-03-30 20:55:08 +00:00
"../src/writer/hlsl/generator_impl_workgroup_var_test.cc",
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
"../src/writer/hlsl/test_helper.h",
]
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
deps = [
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
":tint_unittests_ast_src",
main: Replace --dawn-validation with --validate Performs output validation with spirv-val for SPIR-V (as before), HLSL validation with DXC, and MSL validation with the Metal Shader Compiler. Disable HLSL tests that fail to validate Bug: tint:812 Change-Id: If78c351b4e23c7fb50d333eacf9ee7cc81d18564 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/51280 Reviewed-by: Antonio Maiorano <amaiorano@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Antonio Maiorano <amaiorano@google.com>
2021-06-02 21:02:34 +00:00
"${tint_root_dir}/src:libtint_hlsl_writer_src",
]
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
}
Implement GLSL writer backend. This is a modified version of the HLSL writer. Basic types, arrays, entry points, reserved keywords, uniforms, builtin uniforms, structs, some builtin functions, zero initialization are implemented. Textures, SSBOs and storage textures in particular are unimplemented. All the unit tests "pass", but the output is not correct in many cases. triangle.wgsl outputs correct vertex and fragment shaders that pass GLSL validation via glslang. compute_boids.wgsl outputs a valid but not correct compute shader. Change-Id: I96c7aaf60cf2d4237e45d732e5f51b345aea0552 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/57780 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-10-06 18:55:10 +00:00
tint_unittests_source_set("tint_unittests_glsl_writer_src") {
sources = [
"../src/transform/glsl_test.cc",
"../src/writer/glsl/generator_impl_array_accessor_test.cc",
"../src/writer/glsl/generator_impl_assign_test.cc",
"../src/writer/glsl/generator_impl_binary_test.cc",
"../src/writer/glsl/generator_impl_bitcast_test.cc",
"../src/writer/glsl/generator_impl_block_test.cc",
"../src/writer/glsl/generator_impl_break_test.cc",
"../src/writer/glsl/generator_impl_call_test.cc",
"../src/writer/glsl/generator_impl_case_test.cc",
"../src/writer/glsl/generator_impl_cast_test.cc",
"../src/writer/glsl/generator_impl_constructor_test.cc",
"../src/writer/glsl/generator_impl_continue_test.cc",
"../src/writer/glsl/generator_impl_discard_test.cc",
"../src/writer/glsl/generator_impl_function_test.cc",
"../src/writer/glsl/generator_impl_identifier_test.cc",
"../src/writer/glsl/generator_impl_if_test.cc",
"../src/writer/glsl/generator_impl_import_test.cc",
"../src/writer/glsl/generator_impl_intrinsic_test.cc",
"../src/writer/glsl/generator_impl_intrinsic_texture_test.cc",
"../src/writer/glsl/generator_impl_loop_test.cc",
"../src/writer/glsl/generator_impl_member_accessor_test.cc",
"../src/writer/glsl/generator_impl_module_constant_test.cc",
"../src/writer/glsl/generator_impl_return_test.cc",
"../src/writer/glsl/generator_impl_sanitizer_test.cc",
"../src/writer/glsl/generator_impl_switch_test.cc",
"../src/writer/glsl/generator_impl_test.cc",
"../src/writer/glsl/generator_impl_type_test.cc",
"../src/writer/glsl/generator_impl_unary_op_test.cc",
"../src/writer/glsl/generator_impl_variable_decl_statement_test.cc",
"../src/writer/glsl/generator_impl_workgroup_var_test.cc",
"../src/writer/glsl/test_helper.h",
]
deps = [
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
":tint_unittests_ast_src",
":tint_unittests_transform_src",
Implement GLSL writer backend. This is a modified version of the HLSL writer. Basic types, arrays, entry points, reserved keywords, uniforms, builtin uniforms, structs, some builtin functions, zero initialization are implemented. Textures, SSBOs and storage textures in particular are unimplemented. All the unit tests "pass", but the output is not correct in many cases. triangle.wgsl outputs correct vertex and fragment shaders that pass GLSL validation via glslang. compute_boids.wgsl outputs a valid but not correct compute shader. Change-Id: I96c7aaf60cf2d4237e45d732e5f51b345aea0552 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/57780 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-10-06 18:55:10 +00:00
"${tint_root_dir}/src:libtint_glsl_writer_src",
]
}
BUILD.gn: Fix gn check error The 'tint_unittests_src' source_set needed to depend on gmock_and_gtest, which is normally done with the 'tint_unittests_source_set' template, but this is the root source_set for the unit tests. To fix, these root-level sources have been moved to 'tint_unittests_core_src' which uses the 'tint_unittests_source_set' template. Change-Id: I0d76c55c744646fd7deff22406a668f8c6cdf519 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69220 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Sarah Mashayekhi <sarahmashay@google.com> Kokoro: Ben Clayton <bclayton@google.com>
2021-11-12 11:33:17 +00:00
tint_unittests_source_set("tint_unittests_core_src") {
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
sources = [
"../src/block_allocator_test.cc",
"../src/castable_test.cc",
"../src/clone_context_test.cc",
"../src/debug_test.cc",
"../src/demangler_test.cc",
"../src/intrinsic_table_test.cc",
"../src/program_builder_test.cc",
"../src/program_test.cc",
"../src/scope_stack_test.cc",
"../src/symbol_table_test.cc",
"../src/symbol_test.cc",
"../src/traits_test.cc",
]
Add tests for fuzzers::RandomGenerator BUG=tint:1019 Change-Id: Ia462080877a97348c5589bfa71231a832a7ebfd3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70081 Auto-Submit: Ryan Harrison <rharrison@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-24 14:55:05 +00:00
deps = [ ":tint_unittests_ast_src" ]
}
if (build_with_chromium) {
tint_unittests_source_set("tint_unittests_fuzzer_src") {
sources = [ "../fuzzers/random_generator_test.cc" ]
deps = [
":tint_unittests_core_src",
"${tint_root_dir}/fuzzers:tint_fuzzer_common_src",
]
}
BUILD.gn: Fix gn check error The 'tint_unittests_src' source_set needed to depend on gmock_and_gtest, which is normally done with the 'tint_unittests_source_set' template, but this is the root source_set for the unit tests. To fix, these root-level sources have been moved to 'tint_unittests_core_src' which uses the 'tint_unittests_source_set' template. Change-Id: I0d76c55c744646fd7deff22406a668f8c6cdf519 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69220 Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Sarah Mashayekhi <sarahmashay@google.com> Kokoro: Ben Clayton <bclayton@google.com>
2021-11-12 11:33:17 +00:00
}
source_set("tint_unittests_src") {
testonly = true
deps = [
":tint_unittests_ast_src",
":tint_unittests_core_src",
BUILD.gn: Split up unit tests into separate source sets Prevents errors when we have the same _test.cc file in two different directories. Change-Id: I62eaea9452762670b7a24cdb2d7b0bef4fe52280 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/69102 Kokoro: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-11 16:23:55 +00:00
":tint_unittests_diagnostic_src",
":tint_unittests_inspector_src",
":tint_unittests_resolver_src",
":tint_unittests_sem_src",
":tint_unittests_transform_src",
":tint_unittests_utils_src",
":tint_unittests_writer_src",
"${tint_root_dir}/src:libtint_wgsl_reader_src",
"${tint_root_dir}/src:libtint_wgsl_writer_src",
]
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
if (tint_build_spv_reader) {
deps += [ ":tint_unittests_spv_reader_src" ]
}
if (tint_build_spv_writer) {
deps += [ ":tint_unittests_spv_writer_src" ]
}
if (tint_build_wgsl_reader) {
deps += [ ":tint_unittests_wgsl_reader_src" ]
}
if (tint_build_wgsl_writer) {
deps += [ ":tint_unittests_wgsl_writer_src" ]
}
if (tint_build_msl_writer) {
deps += [ ":tint_unittests_msl_writer_src" ]
}
if (tint_build_hlsl_writer) {
deps += [ ":tint_unittests_hlsl_writer_src" ]
}
Implement GLSL writer backend. This is a modified version of the HLSL writer. Basic types, arrays, entry points, reserved keywords, uniforms, builtin uniforms, structs, some builtin functions, zero initialization are implemented. Textures, SSBOs and storage textures in particular are unimplemented. All the unit tests "pass", but the output is not correct in many cases. triangle.wgsl outputs correct vertex and fragment shaders that pass GLSL validation via glslang. compute_boids.wgsl outputs a valid but not correct compute shader. Change-Id: I96c7aaf60cf2d4237e45d732e5f51b345aea0552 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/57780 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-10-06 18:55:10 +00:00
if (tint_build_glsl_writer) {
deps += [ ":tint_unittests_glsl_writer_src" ]
}
Add tests for fuzzers::RandomGenerator BUG=tint:1019 Change-Id: Ia462080877a97348c5589bfa71231a832a7ebfd3 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/70081 Auto-Submit: Ryan Harrison <rharrison@chromium.org> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
2021-11-24 14:55:05 +00:00
if (build_with_chromium) {
deps += [ ":tint_unittests_fuzzer_src" ]
}
Split BUILD.gn into per-subdirectory BUILD.gns This will allow Tint's dependent to depend on libtint without GN discovering Tint's test and try to build them. In particular it will help use Tint in Dawn in Skia's standalone build which doesn't have //testing. Bug: dawn:706 Change-Id: Idd28662b89aa75df7704eaae205328dce0b96fef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/44540 Reviewed-by: David Neto <dneto@google.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Auto-Submit: Corentin Wallez <cwallez@chromium.org>
2021-03-15 08:43:11 +00:00
configs += [ ":tint_unittests_config" ]
if (build_with_chromium) {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [ "//build/config/compiler:no_chromium_code" ]
}
}
test("tint_unittests") {
deps = [
":gmock_and_gtest",
":tint_unittests_src",
"${tint_spirv_tools_dir}/:spvtools",
"${tint_spirv_tools_dir}/:spvtools_opt",
"${tint_spirv_tools_dir}/:spvtools_val",
]
deps += [ ":tint_unittests_main" ]
configs += [ ":tint_unittests_config" ]
if (build_with_chromium) {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [ "//build/config/compiler:no_chromium_code" ]
}
testonly = true
}