pthread_setname_np() is not provided by the Fuchsia C library
so do not try to call it in Thread::setName().

Change-Id: I7db37e6da52d665ff6535d735e9a2da5d9845143
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35930
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Tested-by: David Turner <digit@google.com>
Presubmit-Ready: David Turner <digit@google.com>

A solution to blocking, non-yielding function calls.

Bug: b/139010488
Change-Id: I8f2a9458980f83fbecafb22a5531aa6e3dbbc040
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35573
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

This finishes our KHR_swapchain v70 implementation.

Test: dEQP-VK.wsi.*
Change-Id: I512e0c086db687ca7ffb8cd5e5375fa7a9d432cd
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35929Tested-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>

Previous versions of dEQP never called this function.

Test: dEQP-VK.wsi.*.swapchain.render.*
Test: dEQP-VK.api.version_check.entry_points
Change-Id: I3d64403984ff60fea9ef2921f7f53f0ca4b94240
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35928Tested-by: Chris Forbes <chrisforbes@google.com>
Presubmit-Ready: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>

Change-Id: Ie970829b56e46ffd1a6cc6028148ecd46d0f098d
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35908
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

A few changes were made to make this work:
- activeLaneMask now contains cMask. Since it's only used in
  loads, stores and atomics, it should be fine to include it
- Added a storesAndAtomicsMask, which also contains sMask
  (stencil) and zMask (depth) for early fragment tests.
  The mask affects all atomic operations and store operations
  into storage buffer and images.
- support for spv::BuiltInHelperInvocation was added

Bug b/140294254

Test: dEQP-VK.*

Change-Id: I42b97a766ddfe331bb2767d80d4360104a221482
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/34114
Presubmit-Ready: Alexis Hétu <sugoi@google.com>
Tested-by: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>

This fixes a few missing dependencies for the Chromium uses of
SwiftShader. It will allow us to turn on the 'gn check' in
Chromium.

Bug: angleproject:3876
Bug: b/140251624
Change-Id: I875a59701a4437cf94b503f6430d858a5285a961
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35811Reviewed-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Jamie Madill <jmadill@chromium.org>

Bug: b/139687059
Change-Id: Ie0084a5f320f2ac36d55ff0950812a77401f4741
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35848Tested-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>

Tested with ANGLE running dEQP-GLES2.*rbo_rgb5_a1*

Bug: b/139411772
Change-Id: I5681b008ae7bfb2df2705189f28667dac8d4c983
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35812Reviewed-by: Nicolas Capens <nicolas.capens@gmail.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Change-Id: Id38fc9787e37108d723edcac73711dc801c46403
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35889
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>

Commands:
git subtree pull --prefix third_party/SPIRV-Tools https://github.com/KhronosGroup/SPIRV-Tools 65e362b7ae2acb8aa5bd2ad516fb793961e673ee --squash

Bug: b/123642959
Change-Id: Idf19b4f893af1967c97d03e984abd1beb14622ca

65e362b7a AggressiveDCEPass: Set modified to true when appending to to_kill_ (#2825)
d67130cac Replace SwizzleInvocationsAMD extended instruction. (#2823)
ad71c057c Replace SwizzleInvocationsMaskedAMD extended instruction. (#2822)
4ae9b7165 Fix gn check (#2821)
35d98be3b Amd ext to khr (#2811)
5a581e738 spvtools::Optimizer - don't assume original_binary and optimized_binary are aliased (#2799)
73422a0a5 Check feature mgr in context consistency check (#2818)
15fc19d09 Refactor instruction folders (#2815)
1eb89172a Add missing files to BUILD.gn (#2809)
8336d1925 Extend reducer to remove relaxed precision decorations (#2797)
b00ef0d26 Handle Id overflow in private-to-local (#2807)
aef8f92b2 Even more id overflow in sroa (#2806)
c5d1dab99 Add name for variables in desc sroa (#2805)
0cbdc7a2c Remove unimplemented method declaration (#2804)
bc62722b8 Handle overflow in wrap-opkill (#2801)
9cd07272a More handle overflow in sroa (#2800)
06407250a Instrument: Add support for Buffer Device Address extension (#2792)
7b4e5bd5e Update remquo validation to match the OpenCL Extended Instruction Set Specification (#2791)
dac9210dc Use ascii code based characters (#2796)
ff872dc6b Change the way to include header (#2795)
bbd80462f Fix validation of constant matrices (#2794)
60043edfa Replace OpKill With function call. (#2790)
f701237f2 Remove useless semi-colons (#2789)
95386f9e4 Instrument: Fix version 2 output record write for tess eval shaders. (#2782)
22ce39c8e Start SPIRV-Tools v2019.5
d65513e92 Finalize SPIRV-Tools v2019.4
4b64beb1a Add descriptor array scalar replacement (#2742)
c26c2615f Update CHANGES
29af42df1 Add SPV_EXT_physical_storage_buffer to opt whitelists (#2779)
b029d3697 Handle RelaxedPrecision in SROA (#2788)
370375d23 Add -fextra-semi to Clang builds (#2787)
698b56a8f Add 'copy object' transformation (#2766)
4f14b4c8c fuzz: change output extension and fix usage string (#2778)
0b70972a2 Remove extra ';' after member function definition. (#2780)
5ada98d0b Update WebGPU validation rules of OpAtomic*s (#2777)
3726b500b Treat access chain indexes as signed in SROA (#2776)
31590104e Add pass to inject code for robust-buffer-access semantics (#2771)
4a28259cc Update OpMemoryBarriers rules for WebGPU (#2775)
7621034aa Add opt test fixture method SinglePassRunAndFail (#2770)
ac3d13105 Element type is const for analysis::Vector,Matrix,RuntimeArray (#2765)
49797609b Protect against out-of-bounds references when folding OpCompositeExtract (#2774)
7fd2365b0 Don't move debug or decorations when folding (#2772)
7bafeda28 Update OpControlBarriers rules for WebGPU (#2769)

git-subtree-dir: third_party/SPIRV-Tools
git-subtree-split: 65e362b7ae2acb8aa5bd2ad516fb793961e673ee

Instead of using a batch size of 1, divide the batch size by 3 so that
we have enough room for 3x more primitives to render. Also use local
Triangle data structures to copy the vertices for the extra primitives,
instead of adding them to the input batch.

Bug: b/139872671
Change-Id: I1bc860d291b53fdd266b6c018ac0c47c876aaa09
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35588Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Implements Vulkan's 'fillModeNonSolid' feature.

Mostly a copy-and-paste of src/Renderer logic. It sets the batch size
for non-solid triangles to 1 so their edges or vertices can get
individually rasterized as lines or points.

Bug: b/139872671
Tests: dEQP-VK.rasterization.culling.*
Change-Id: Ifbae8dad32e8b43159fee459c75d7e593fc08041
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35031Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

LLVM takes liberties with NaN float constants, which can arise in integer
constant values.

Bug b/140294254

Test: dEQP-VK.spirv_assembly.type.vec3.u32.constant_composite_frag
Change-Id: Ibfff77b9e0bacf81756bb50801615369dfc51ab6
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35728
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

Change-Id: Ie2c5c11c029ddd41944ef2def2560176ebd244b7
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35729
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>

https://swiftshader-review.googlesource.com/c/SwiftShader/+/35630
renamed the version scripts with corresponding changes in CMake, but
other build files were overlooked.

Bug: b/139189696
Change-Id: Ib6354e0fb4f62d1e7f4d4f335d03b45ad205c87e
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35808
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

Windows DLLs are not systematically prefixed with "lib".
This change follows the implicit and explict expectations set out in:
https://vulkan.lunarg.com/doc/view/1.0.21.0/windows/LoaderAndLayerInterface.html
http://tldp.org/HOWTO/Program-Library-HOWTO/shared-libraries.html#AEN49

Also make the vk_swiftshader target a dependency of vk-unittests so that
building the latter also allows running it.

Bug: b/139189696
Change-Id: I070871e63c5cefc3f80a6f441a1d5097c85b82b6
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35630
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

This CL enables the Windows build integration with ANGLE. Several
instances of "build_with_chromium" in the GN are changed to use
build_overrides instead. Also in several places in the GN support
is added for Windows builds.

Also Add guards for WIN32_LEAN_AND_MEAN, and removes an unused
variable that shows up when building with GN on Windows configs.

Bug: angleproject:3876
Bug: b/140251624
Change-Id: I7a470c7605c2b62420dca44e4797c29983ed7c13
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35631Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Jamie Madill <jmadill@chromium.org>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

The BlendState structure is part of the cache key used to look up pixel
routines, which must not contain any uninitialized bits. The
PixelProcessor::States structure gets zero-initialized as a whole by
the Memset<> base class, but when writing BlendState structures into it
any uninitialized bits may get copied over.

Thus BlendState must also be zero-initialized, using Memset<>.

Bug: b/140286664
Bug: b/140193782
Change-Id: I031ec1a5f3be4329ae16366bfd4c8f17e2371384
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35768Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>

ANGLE needs to be able to sample surfaces in
order to emulate things like glBlitFramebuffer
or use EGL images as textures.

Section 32.3. Required Format Support of the
Vulkan 1.1 spec dictates that all formats
which must support
VK_FORMAT_FEATURE_COLOR_ATTACHMENT_BIT,
also support
VK_IMAGE_USAGE_TRANSFER_SRC_BIT,
VK_IMAGE_USAGE_TRANSFER_DST_BIT,
and VK_IMAGE_USAGE_SAMPLE_BIT.

It should already work properly in SwiftShader,
so this cl just adds the bit.

Bug b/139412450 b/139479506

Change-Id: I0f79d042789a8b6372f75ecf6cda5f28c6955652
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35709Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Alexis Hétu <sugoi@google.com>

A few blend related context members were changed from scalars to
arrays in order to support independent blend.

Bug b/140193782

Change-Id: I5ca1153e952fe0d3899f68dc6cd7cc5d8a244b72
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/34113Tested-by: Alexis Hétu <sugoi@google.com>
Presubmit-Ready: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>

The Visual Studio 2017 solution will be deprecated in favor of using
Visual Studio 2019 for building the CMake project.

For now we're keeping it for just the legacy OpenGL ES and Direct3D
builds, but Vulkan related projects are removed since they break the
full solution build frequently and fixing them may be more involved than
running build/cmake.sh again.

Bug: swiftshader:134
Bug: b/139189696
Change-Id: I5225a291c6cf4bb8c5d0b9a2f7a16f2e85823feb
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35708
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

Reduce the test timeout to 2 minutes, 10 is excessive.

Include the stderr / stdout in the error message when a test crashes.

Up the max proc memory limit from 3GB to 4GB. While our tests haven't started using dramatically more memory, Golang (which wraps the process) seems to use more recently.

Reduce delay in killing a timedout process.

Change-Id: I4634a788a5cae2cb088d70d2f35909a0264600d6
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35692Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Much like XlibSurfaceKHR, doesn't cope with any formats aside from 24-bit RGB.

Bug: b/124265819
Bug: b/139412276
Bug: b/139491466
Change-Id: I1146f6ff672a9016be04e6b4d595de40fa03346f
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35149
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Change-Id: I33a38b285c02ce33fe17e662fd9be93fda97f6d5
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35689Reviewed-by: Chris Forbes <chrisforbes@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Storage image support for these formats is mandatory, but this was not
covered in previous versions of CTS.

Bug: b/140208021
Test: dEQP-VK.image.format_reinterpret.*
Change-Id: I823e6ebf02b0aee4d3d33f12710304f8eb922d19
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35669Tested-by: Chris Forbes <chrisforbes@google.com>
Presubmit-Ready: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>

MemorySanitizer has no visibility into writes performed by JIT'd code.

If we have built the project with the MemorySanitizer enabled, memset the Triangle and Primitive structs so the sanitizer has a write to see.

This is only cleared in MemorySanitizer enabled builds, as this adds a non negligable cost to Renderer construction.

Bug: chromium:998457
Change-Id: I3b4f9e48783c6e8a1109a578988567b0a42eb1f9
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35688
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>

Per Vulkan spec 2.9.2, when converting from a floating point value to
unsigned normalized, we must first clamp to [0,1]. When converting from
a floating point value to signed normalized, we must first clamp to
[-1,1]. This wasn't done to clear values.

Test: dEQP-VK.rasterization.provoking_vertex.*
Bug: b/140203285
Change-Id: I3f8c421b187b6a7e8bcb22d45142dc1664221605
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35668Tested-by: Chris Forbes <chrisforbes@google.com>
Presubmit-Ready: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>

This also fixes running them from within Visual Studio without requiring
setting any environment variables.

Change-Id: I5592b2d6a0abfeaeffbe1b788529c77adf5764ed
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35629
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>

This wasn't ever called by previous versions of deqp.

Test: dEQP-VK.wsi.xlib.surface.query_devgroup_present_modes
Bug: b/140182247
Change-Id: Id77bdc726681922b99f3fa8d098cad7853293d29
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35648Tested-by: Chris Forbes <chrisforbes@google.com>
Presubmit-Ready: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Bug: b/140178640
Test: dEQP-VK.renderpass.suballocation.unused*
Change-Id: I3ad3ef55946e4f63e941e6df11a02ca4e2ac6e26
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35554Tested-by: Chris Forbes <chrisforbes@google.com>
Presubmit-Ready: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

Change-Id: I0c457c4a128180b804dddb736951612a094862b6
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35574Reviewed-by: Chris Forbes <chrisforbes@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>

Change-Id: I1230607ebbf0e7dadf8603d2fde5e7a95441c713
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35628
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>

Change-Id: I6c50a3626297aaec7a249068687643f53da4b475
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35553Reviewed-by: Ben Clayton <bclayton@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Caught by Chromium's MSAN tests:
https://chromium-review.googlesource.com/c/chromium/src/+/1774264/3#message-8031b1ce1959519ccdbfa83113f0d989560148d5

Bug: b/139010488
Bug: chromium:998457
Change-Id: If14f0777155ab892f29eb9b01c0bc6781510381b
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35575Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Addresses TSAN error in Chromium viz_unittests and cc_unittests.

Bug: b/139010488
Change-Id: I1a53487c4787170c38898cca47bf76145ebdf6bd
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35608
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>

Drop the complex task scheduling logic for yarn.
Performance gains seen up to around ~30% FPS.

Bug: b/139142453
Change-Id: I264fee36323425a791088565d99dc586670a948a
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35572Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>

Pass the cluster count to the QuadRasterizer instead of using a rather nasty back dependency to the renderer.

This allows the cluster count to be adjusted per-draw without requiring synchronization around the count.

The primary goal of this change is that we can scale the number of clusters based on the complexity of the draw.

Bug: b/139142453
Change-Id: I0379e16568de402f186ee2dd1e8b2346bed30efd
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35571Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>

These were part of a larger stack of changes for Yarn, but aren't specific to anything in particular.

Change-Id: I863d8bbb316b5c255be9557b83511bb9cdef70b0
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35570Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>