From 66794eada3b0788d9cbc3a8c862b29028378d2ee Mon Sep 17 00:00:00 2001 From: David Neto Date: Wed, 25 Nov 2020 21:13:17 +0000 Subject: [PATCH] Roll dependencies; spirv-reader: avoid spurious OpLine Roll third_party/gpuweb-cts/ 2a53a9bd3..abe6e1dd2 (51 commits) https://chromium.googlesource.com/external/github.com/gpuweb/cts.git/+log/2a53a9bd3d45..abe6e1dd262d $ git log 2a53a9bd3..abe6e1dd2 --date=short --no-merges --format='%ad %ae %s' 2020-11-25 corentin Port the robust vertex access tests from GLSL to WGSL (#379) 2020-11-25 corentin Port createRenderPipeline validation tests from GLSL to WGSL (#372) 2020-11-23 kainino iterateCollapsedQueries: don't yield files with no tests (#383) 2020-11-20 kainino Clean trailing whitespace inside shader strings (#382) 2020-11-20 yunchao.he Memory sync test for buffer (#302) 2020-11-20 corentin Port verteState validation tests from GLSL to WGSL (#376) 2020-11-20 corentin Port the setVertexBuffer validation tests from GLSL to WGSL (#375) 2020-11-20 corentin Port the render pass validation tests from GLSL to WGSL (#374) 2020-11-20 corentin Port the validation index access tests from GLSL to WGSL (#373) 2020-11-20 corentin Group and update the dynamic state validation tests. (#307) 2020-11-20 corentin Port the primitive topology tests from GLSL to WGSL (#371) 2020-11-20 corentin Port culling tests from GLSL to WGSL (#370) 2020-11-20 corentin Cleanups in webgpu/api/operation/command_buffer (#369) 2020-11-20 kainino From HackMD plan, part 2: some more validation tests (#368) 2020-11-19 kainino Fix cross-device fence signal tests (cherry-pick) (#366) 2020-11-19 kainino From HackMD plan: manual, stress, web_platform, documentation, and some validation (#364) 2020-11-18 corentin Port basic command_buffer tests from the GLSL branch (#350) 2020-11-17 kainino add src/webgpu as a separate folder in the workspace 2020-11-17 kainino Replace grunt tasks with shell tasks 2020-11-17 kainino Check directory names match validQueryPart 2020-11-17 kainino rename web-platform to web_platform 2020-11-17 2154796+austinEng Remove TexelDataRepresentationImpl.isGPULittleEndian (#357) 2020-11-17 2154796+austinEng Implement texelData for rgb9e5ufloat and add texelData unit tests (#352) 2020-11-17 2154796+austinEng Refactor texture zero init tests and use WGSL (#353) 2020-11-17 corentin Port the renderPass resolve tests from GLSL to WGSL (#351) 2020-11-13 mehmetoguzderin Make comments optional (#348) 2020-11-12 corentin Add gen_wpt_cts_html as an NPM script (#347) 2020-11-11 9856269+sarahM0 [wgsl] Update tests to match spec update: Relaxing void return requirement (#330) 2020-11-11 mehmetoguzderin Check PRs to glsl-dependent (#341) 2020-11-10 kainino Remove travis config from main branch (#336) 2020-11-10 kainino Remove AsyncMutex 2020-11-09 kainino Simplify WPT runtime 2020-11-10 mehmetoguzderin Fix Actions on CI (#334) 2020-11-10 corentin Roll webgpu-types. (#329) 2020-11-10 mehmetoguzderin Add CI on Actions (#331) 2020-11-05 natashalee add create sampler validation test (#327) 2020-11-04 kainino wpt: Don't return promise from async_test 2020-11-04 kainino Allow marking tests as explicitly unimplemented (#319) 2020-11-04 kainino standalone: Don't generate DOM when test finishes (#323) 2020-11-04 kainino standalone: Show only the full query string for a test, and aligned (#324) 2020-11-03 kainino standalone: Replace results JSON textarea with copy button (#322) 2020-11-02 kainino standalone: Lazily-generate DOM subtrees (#321) 2020-10-27 enrico.galli Add API validation tests for compute pass (#308) 2020-10-26 kainino Optimize device pool lookup (#295) 2020-10-23 kainino Remove unneeded @babel/plugin-proposal-class-properties (#313) 2020-10-23 kainino Remove non-test dependence on WebGPU ambient constants (#318) 2020-10-20 kainino Loop to exhaust eventualExpectations (#309) 2020-10-20 dsinclair Update WGSL to new struct syntax. (#316) 2020-10-19 kainino Add support for logical assignment operators (&&= ||=) (#311) 2020-10-19 kainino Update npm dependencies (#310) 2020-10-20 corentin Add validation tests for mapAsync. (#301) Created with: roll-dep third_party/gpuweb-cts Change-Id: Ibed486c6e02d8df9db7a66c00c4546c46df0c1fc Roll third_party/spirv-tools/ 7e1825a59..2c458414c (62 commits) https://chromium.googlesource.com/external/github.com/KhronosGroup//SPIRV-Tools.git/+log/7e1825a59619..2c458414c085 $ git log 7e1825a59..2c458414c --date=short --no-merges --format='%ad %ae %s' 2020-11-25 dneto BuildModule: optionally avoid adding new OpLine instructions (#4033) 2020-11-25 dneto Remove prototype for unimplemented method (#4031) 2020-11-25 afdx spirv-fuzz: Fix facts arising from CompositeConstruct (#4034) 2020-11-24 afdx spirv-fuzz: Do not flatten conditionals that create synonyms (#4030) 2020-11-23 dneto Update MeshShadingNV dependencies (and land Ray tracing updates) (#4028) 2020-11-18 greg Fix buffer oob instrumentation for matrix refs (#4025) 2020-11-13 afdx spirv-opt: Set parent when adding basic block (#4021) 2020-11-13 jaebaek spirv-opt: properly preserve DebugValue indexes operand (#4022) 2020-11-11 dneto Use less stack space when validating Vulkan builtins (#4019) 2020-11-05 46493288+sfricke-samsung spirv-val: Fix SPV_KHR_fragment_shading_rate VUID label (#4014) 2020-11-05 46493288+sfricke-samsung spirv-val: Label Layer and ViewportIndex VUIDs (#4013) 2020-11-05 alanbaker Add dead function elimination to -O (#4015) 2020-11-04 jaebaek Add DebugValue for invisible store in single_store_elim (#4002) 2020-11-04 dnovillo Fix SSA re-writing in the presence of variable pointers. (#4010) 2020-11-04 afdx spirv-fuzz: Fixes to pass management (#4011) 2020-11-03 afdx spirv-fuzz: Add support for reining in rogue fuzzer passes (#3987) 2020-11-03 vasniktel spirv-fuzz: Fix assertion failure in FuzzerPassAddCompositeExtract (#3995) 2020-11-03 vasniktel spirv-fuzz: Fix invalid equation facts (#4009) 2020-11-03 vasniktel spirv-fuzz: Fix bugs in TransformationFlattenConditionalBranch (#4006) 2020-11-03 andreperezmaselco.developer spirv-fuzz: Fix bug related to transformation applicability (#3990) 2020-10-30 jaebaek Temporarily add EmptyPass to prevent glslang from failing (#4004) 2020-10-30 Junda.Liu spirv-opt: Add support to prevent functions from being inlined if they have DontInline flag (#3858) 2020-10-29 jaebaek Propagate OpLine to all applied instructions in spirv-opt (#3951) 2020-10-29 bclayton CMake: Add SPIRV_TOOLS_BUILD_STATIC flag (#3910) 2020-10-29 dneto Avoid copying a ref in a loop (#4000) 2020-10-28 justsid spirv-val: Allow the ViewportIndex and Layer built-ins on SPIR-V 1.5 (#3986) 2020-10-28 dnovillo Simplify logic to decide whether CCP modified the IR (#3997) 2020-10-27 jaebaek Add DebugValue for DebugDecl invisible to value assignment (#3973) 2020-10-26 greg Fix bounds check instrumentation to handle 16-bit values (#3983) 2020-10-23 andreperezmaselco.developer spirv-fuzz: Add expand vector reduction transformation (#3869) 2020-10-23 vasniktel spirv-fuzz: Don't replace irrelevant indices in OpAccessChain (#3988) 2020-10-23 vasniktel spirv-fuzz: Add FuzzerPassAddCompositeExtract (#3904) 2020-10-22 afdx spirv-fuzz: Fix mismatch with shrinker step limit (#3985) 2020-10-22 afdx spirv-fuzz: Fix off-by-one error in replayer (#3982) 2020-10-22 afdx spirv-fuzz: Get order right for OpSelect arguments (#3974) 2020-10-22 afdx spirv-fuzz: Do not add synonym-creating loops in dead blocks (#3975) 2020-10-22 afdx spirv-fuzz: Skip OpTypeSampledImage when propagating up (#3976) 2020-10-22 afdx spirv-fuzz: Pass OpUndef in function call if needed (#3978) 2020-10-22 afdx spirv-fuzz: Fix off-by-one in TransformationCompositeConstruct (#3979) 2020-10-22 afdx spirv-fuzz: Tolerate absent ids in data synonym fact management (#3966) 2020-10-21 afdx spirv-fuzz: Fix to id availability (#3971) 2020-10-21 afdx spirv-fuzz: Fix operand types (#3962) 2020-10-21 8729214+jonahryandavis Update SPIRV-Headers revision in DEPS file (#3961) 2020-10-21 afdx spirv-fuzz: Don't flatten conditional if condition is irrelevant (#3944) 2020-10-21 afdx spirv-fuzz: Do not produce OpPhis of type OpTypeSampledImage (#3964) 2020-10-21 afdx spirv-fuzz: Restrict fuzzer pass to reachable blocks (#3970) 2020-10-21 afdx spirv-fuzz: Handle more types when extending OpPhi instructions (#3969) 2020-10-21 afdx spirv-fuzz: Skip early terminator wrappers when merging returns (#3968) 2020-10-21 afdx spirv-fuzz: Avoid irrelevant constants in synonym-creating loops (#3967) 2020-10-21 afdx spirv-fuzz: Skip dead blocks in FuzzerPassAddOpPhiSynonyms (#3965) 2020-10-21 afdx spirv-fuzz: Avoid the type manager when looking for struct types (#3963) 2020-10-20 afdx spirv-fuzz: Fix to TransformationDuplicateRegionWithSelection (#3941) 2020-10-20 afdx spirv-fuzz: Skip OpFunction when replacing irrelevant ids (#3932) 2020-10-20 afdx spirv-fuzz: Use component-wise selectors when flattening conditional branches (#3921) 2020-10-20 TobyHector Add SPV_EXT_shader_image_int64 (#3852) 2020-10-20 TobyHector Support SPV_KHR_fragment_shading_rate (#3943) 2020-10-19 afdx spirv-val: Fix validation of OpPhi instructions (#3919) 2020-10-19 afdx spirv-fuzz: Avoid void struct member when outlining functions (#3936) 2020-10-19 afdx spirv-fuzz: Do not allow Block-decorated structs when adding parameters (#3931) 2020-10-19 afdx spirv-fuzz: Fix to operand id type (#3937) 2020-10-19 afdx spirv-fuzz: Handle dead blocks in TransformationEquationInstruction (#3933) 2020-10-19 afdx spirv-fuzz: Do not allow sampled image load when flattening conditionals (#3930) Created with: roll-dep third_party/spirv-tools Change-Id: Ic75967a4b775dfae8b53a679f48bcc8b3f162217 Roll third_party/spirv-headers/ c43a43c7c..104ecc356 (9 commits) https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Headers.git/+log/c43a43c7cc3a..104ecc356c1b $ git log c43a43c7c..104ecc356 --date=short --no-merges --format='%ad %ae %s' 2020-11-12 dneto MeshShadingNV enables builtins PrimitiveId, Layer, and ViewportIndex 2020-10-16 dkoch de-alias/reassign OpIgnoreIntersectionKHR/OpTerminateRayKHR 2020-06-29 alele Raytracing and Rayquery updates for final 2020-06-15 alele Updated headers for new trace/executeCallable and acceleration structure cast. 2020-11-04 michael.kinsner Reserve additional loop control bit for Intel extension (NoFusionINTEL) (#175) 2020-11-02 4464295+XAMPPRocky Add EmbarkStudios/rust-gpu to vendor list. (#174) 2020-10-23 john Bump revision to 4, for SPIR-V 1.5. 2020-10-19 TobyHector Add SPV_EXT_shader_image_int64 (#170) 2020-10-19 TobyHector Added SPV_KHR_fragment_shading_rate (#172) Created with: roll-dep third_party/spirv-headers Change-Id: Ib43e5574d9fc1d2cffdd3a6cf66cbc3a88db2051 Roll third_party/googletest/ 3c95bf552..b1fbd33c0 (30 commits) https://chromium.googlesource.com/external/github.com/google/googletest.git/+log/3c95bf552405..b1fbd33c06cd $ git log 3c95bf552..b1fbd33c0 --date=short --no-merges --format='%ad %ae %s' 2020-11-23 absl-team Googletest export 2020-11-19 absl-team Googletest export 2020-11-13 vlee Initialize TestInfo member is_in_another_shard_ in constructor. 2020-11-12 absl-team Googletest export 2020-11-12 absl-team Googletest export 2020-11-12 absl-team Googletest export 2020-11-11 dmauro Googletest export 2020-11-11 dmauro Googletest export 2020-11-11 dmauro Googletest export 2020-11-11 absl-team Googletest export 2020-11-11 marius.brehler Refactor finding python 2020-11-06 absl-team Googletest export 2020-11-06 absl-team Googletest export 2020-10-29 knut Only save original working directory if death tests are enabled 2020-11-08 hyuk.myeong fix typos 2020-11-06 absl-team Googletest export 2020-11-05 ofats Googletest export 2020-10-27 absl-team Googletest export 2020-10-27 elliott.brossard Add instructions for sanitizer integration 2020-10-26 absl-team Googletest export 2020-10-20 sonzogniarthur Fix typo "definedin in" => "defined in" 2020-10-15 absl-team Googletest export 2020-10-15 absl-team Googletest export 2020-10-07 manavrion Improve FilePath::Normalize method 2020-09-16 hyuk.myeong Remove spaces between Google Test and Google Mock 2020-09-15 hyuk.myeong Add follow-up patch for more natural reading 2020-09-15 hyuk.myeong Apply the reviewed comment 2020-09-15 hyuk.myeong Remove a space 2020-09-15 hyuk.myeong Improve the tutorial that may be confusing 2020-02-17 krystian.kuzniarek remove a duplicated include Created with: roll-dep third_party/googletest Change-Id: I62335cb23a96218f62c1842e786777d7e8dfe4f2 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/34007 Reviewed-by: Ryan Harrison Commit-Queue: David Neto Auto-Submit: David Neto --- DEPS | 8 ++++---- src/reader/spirv/parser_impl.cc | 4 +++- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/DEPS b/DEPS index 2fa9ad975f..23ca99236d 100644 --- a/DEPS +++ b/DEPS @@ -10,10 +10,10 @@ vars = { 'buildtools_revision': 'ff93f3ea1a7f033d3caf8f60ec1937cc71351419', 'clang_revision': 'fcef86e30a0ab061b982b5c9d91bb060df8f5269', 'cpplint_revision': '305ac8725a166ed42e3f5dd3f80d6de2cf840ef1', - 'googletest_revision': '3c95bf552405fd0cc63cea0ca2f6c4cd89c8d356', - 'gpuweb_cts_revision': '2a53a9bd3d45444e1d101ca535367ba6fa15150a', - 'spirv_headers_revision': 'c43a43c7cc3af55910b9bec2a71e3e8a622443cf', - 'spirv_tools_revision': '7e1825a59619f3decd9b46f25d457b1a8878ca89', + 'googletest_revision': 'b1fbd33c06cdb0024c67733c6fdec2009d17b384', + 'gpuweb_cts_revision': 'abe6e1dd262d96749966027fdf20eb4a34710ba1', + 'spirv_headers_revision': '104ecc356c1bea4476320faca64440cd1df655a3', + 'spirv_tools_revision': '2c458414c0851b9a0d1b0493857b56a1089847ac', 'testing_revision': '2691851e49de541c3fe42fa8692ddcdee938162f', } diff --git a/src/reader/spirv/parser_impl.cc b/src/reader/spirv/parser_impl.cc index 4ea1e21e6e..f41c4ed473 100644 --- a/src/reader/spirv/parser_impl.cc +++ b/src/reader/spirv/parser_impl.cc @@ -431,8 +431,10 @@ bool ParserImpl::BuildInternalModule() { } const spv_context& context = tools_context_.CContext(); + const bool insert_extra_opline = false; ir_context_ = spvtools::BuildModule(context->target_env, context->consumer, - spv_binary_.data(), spv_binary_.size()); + spv_binary_.data(), spv_binary_.size(), + insert_extra_opline); if (!ir_context_) { return Fail() << "internal error: couldn't build the internal " "representation of the module";