Commands:
git subtree pull --prefix third_party/SPIRV-Headers https://github.com/KhronosGroup/SPIRV-Headers master --squash

Bug: b/123642959
Change-Id: I5c51e8aa9dcdd79b80749ce11c5697e84361c6a0

af64a9e82 Versioning: Complete the versioning change in recent commits.
5f3a03023 Merge pull request #133 from nhaehnle/buildHeader-spv15
809512f36 buildHeaders: update version to SPIR-V 1.5
b252a5095 Merge pull request #131 from lukaszgotszaldintel/new_branch
842ec9067 Merge pull request #128 from amdrexu/bugfix
1cdfc756f Merge pull request #129 from ehsannas/update_doc
7b1b05e78 Improve the doc on using Bazel.
836e001d7 Merge pull request #127 from ehsannas/add_bazel_build
3f9bad6fc add cmake option SPIRV_HEADERS_SKIP_INSTALL
c98d953a3 Bump the SPIR-V version to 1.5
ab22babed Update documentation.
4163b3639 Add a Bazel build file.
601d73872 Add SPV_KHR_physical_storage_buffer.
63d4d272f SPIR-V 1.5.
38cafab37 Minor tweak for Rev. 2 of SPIR-V 1.4.
059a49598 Merge branch 'printing-classes'
cca9cc7f3 Grammar: Add instruction-printing classes.

git-subtree-dir: third_party/SPIRV-Headers
git-subtree-split: af64a9e826bf5bb5fcd2434dd71be1e41e922563

Bug: b/140546382
Change-Id: I71253f73dcd3c2083873cb1f0084b42190e9df8e
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38209Tested-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Removes the last use of sw::Event in Vulkan.

Bug: b/140546382
Change-Id: I5de63c52e35756dc6d7e4c6cbb77669ea3925d79
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38208Tested-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Bug: b/144388391
Change-Id: I5a9c63899fcb1a39c80be25813d1d89ca22b26a8
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38138
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>

First step to using marl::Event::any()

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

* changes:
  Update Marl to 6dd9ff1e
  Squashed 'third_party/marl/' changes from 49e4e3141..14e4d862a

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

Includes new Event methods which can be used for Vulkan fences.

Changes:
    14e4d862a Scheduler: Fix issues with fiber lists a timeouts.
    57f41915d SwiftShader build fixes.
    791187298 Add Event::any().
    ecd5ab322 Implement yields with timeouts, wait_for() / wait_until()
    6ba730d94 Update README.md
    8348be4f0 Implement page-based functions for Fuchsia
    5e512cd0c Fix condition logic of assert in TasksOnlyScheduledOnWorkerThreads.
    37ae48f40 Add marl::Event.
    a90725760 Update README.md (#49)

Commands: git subtree pull --prefix third_party/marl https://github.com/google/marl master --squash

Bug: b/140546382

Change-Id: Ifb21caa23fec31e17d9459f52c7cd0bbcc1e499c

Includes a test fix for dEQP-VK.api.image_clearing.dedicated_allocation.clear_depth_stencil_attachment.multiple_layers.d32_sfloat_s8_uint_separate_layouts_stencil_200x180 incorrectly assuming VK_KHR_create_renderpass2 is supported.

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

14e4d862a Scheduler: Fix issues with fiber lists a timeouts.
57f41915d SwiftShader build fixes.
791187298 Add Event::any().
ecd5ab322 Implement yields with timeouts, wait_for() / wait_until()
6ba730d94 Update README.md
8348be4f0 Implement page-based functions for Fuchsia
5e512cd0c Fix condition logic of assert in TasksOnlyScheduledOnWorkerThreads.
37ae48f40 Add marl::Event.
a90725760 Update README.md (#49)

git-subtree-dir: third_party/marl
git-subtree-split: 14e4d862a959b831fd994a436e7c104c6fd19006

Bug: b/143351714
Change-Id: I8adab60da392ebf143260f244024979ecc5c32eb
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/37609
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>

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

The previous 'connected diamonds' polygon that was used to rasterize
Bresenham lines suffered from duplicate rasterization of endpoints,
which violates the diamond exit-rule and is disallowed by the Vulkan
(and OpenGL) spec.

This change rasterizes Bresenham lines as a parallelogram, as described
by Vulkan's non-strictLines algorithm.

This satisfied Vulkan's requirements laid out in section 26.10.2
Bresenham Line Segment Rasterization:

"Implementations may use other line segment rasterization algorithms,
 subject to the following rules:
 - The coordinates of a fragment produced by the algorithm must not
   deviate by more than one unit in either x or y framebuffer
   coordinates from a corresponding fragment produced by the diamond-
   exit rule.
 - The total number of fragments produced by the algorithm must not
   differ from that produced by the diamond-exit rule by no more than
   one.
 - For an x-major line, two fragments that lie in the same framebuffer-
   coordinate column must not be produced (for a y-major line, two
   fragments that lie in the same framebuffer-coordinate row must not be
   produced).
 - If two line segments share a common endpoint, and both segments are
   either x-major (both left-to-right or both right-to-left) or y-mayor
   (both bottom-to-top or both top-to-bottom), then rasterizing both
   segments must not produce duplicate fragments. Fragments also must
   not be omitted so as to interrupt continuity of the connected
   segments."

OpenGL ES line rasterization has not been modified as part of this
change, to not require rebasing of golden images, but the parallelogram
algorithm was made available for easy comparison.

Bug: b/80135519
Change-Id: I09e8b90a393d3a08387d79669d9dbe5f83a0811d
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38049
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

baseVertex is a signed integer that's allowed have a negative value.
In order to support it, the offset was moved from being applied
on the index buffer memory pointer directly to being applied
to the indices from the index buffer inside the Vertex routine.

Fixes all issues in the following subcategory using SwANGLE:
dEQP-GLES31.functional.draw_indirect.*

Bug: b/144353667
Change-Id: Ia70c258eab25c274553c4955dc2336a42d43f005
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38148
Presubmit-Ready: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>

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

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

This extension allows one to import/export device memory buffers
through shared memory region file descriptors.

This also adds checks to ensure that binding a buffer or image
to an external device memory works only if VkCreate{Buffer,Image}
was called with a VkExternalMemory{Buffer,Image}CreateInfo struct
with compatible handle types.

Test: dEQP-VK.api.external.memory.opaque_fd*

Bug: b/140419396
Change-Id: I5d249685896ae0764bc9d5c635cc3799323db453
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35152
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: David Turner <digit@google.com>

Use go 1.13's new error functionality to create nested errors, and use the new errors.As() to unwrap them.

Fixes bad classification of tests that have recently started returning non-zero exit codes.

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

Update the maxPerStageDescriptorStorageBuffers from 4 to 16.
ANGLE requires at least 12 for GLES3.1 support, and 16 is a common
number among actual hardware so bumping to 16.

Bug: swiftshader:136
Change-Id: I2790a8cab197c773aa14d9c1b0fdcc5ca8356b25
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38068
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Tobin Ehlis <tobine@google.com>

This reverts commit 5d0faf9b.

Reason for revert: Marl issue was fixed; https://github.com/google/marl/issues/38

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

It seems that once a change is no longer in the top N most recent changes, the server will start responding `server does not allow request for unadvertised object` when attempting to fetch the specific change by SHA.

To work around this, we need to fetch the branch by name, then checkout the specific change locally. As the Khronos seems to be particularly slow to respond today, I've limited the branch depth to 99, for now. This hopefully will not be a problem as we should typically only be updating to newer changes.

Also bumped the git command timeout to 15 mintutes as this repo can be _really_ slow.

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

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

For 565 and 5551 formats, the blend equations were getting
applied on top of colors represented by the top 5 or 6 MSB
only, leading to precision errors.

Fixed this by:
- removing the scaling applied on the source color before
  blending is applied and the related inverse transform in
  the writeColor function
- adding the proper remapping of the color to the full color
  range in the readPixel function

That way, we're always working with full 16 bit range colors
and the blend equations work properly.

Fixes all tests in:
dEQP-GLES3.functional.fragment_ops.interaction.basic_shader.*

Bug: angleproject:4016 b/24332884

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

This change allows for outputting the SwiftShader version to Android's
logcat. The version information is as follows:

major.minor.patch.commit_hash

In logcat, when SwiftShader is loaded by ANGLE, this looks like:

11-07 12:17:10.152  9110  9110 I SwiftShader: SwiftShader Version: 5.0.0.9c7bf8093a75
11-07 12:17:10.162  9110  9110 I ANGLE   : Version (2.1.0.a2b1a958e2d8), Renderer (Vulkan 1.1.0(SwiftShader Device (0x0000C0DE)))

This allows developers to verify that they are running the expecting
build of SwiftShader when debugging problems or running conformance
tests.

The version information is logged to logcat when the build flag
"ENABLE_BUILD_VERSION_OUTPUT" is enabled within src/Android.bp.

Bug: b/142828252
Change-Id: Iff773b16a2f3532aa843629ec50519b519bbadac
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/37990
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Tim Van Patten <timvp@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>

Test: dEQP-VK.renderpass.suballocation.unused_clear_attachments.*depth*
Bug: b/144280552
Change-Id: I37072270105caf206ed3ab6cb2895d32ef22f653
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38108Tested-by: Chris Forbes <chrisforbes@google.com>
Presubmit-Ready: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

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

The module may legitimately contain descriptor set references (intended
for another entrypoint) which are not compatible with our limits.

There is a matching assert to verify the set index at the point the
descriptor is actually referenced.

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

Moved some functionality from libVulkan.cpp to VkImage.cpp
in order to be able to remove the global androidSwapchainMap
which may have been causing sync issues.

I tried to make as much code as possible compile on desktop
so that we have a better chance of detecting if a desktop
related change breaks Android specific code.

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

Fixes the loading of the dEQP config file from the CI change.

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

(a7d268a6a3ee9655298d61cdb8b5c42306a4e335)

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

There were two bugs here:

- Across a 2x2 pixel quad, we were producing a uniform PointCoord value.
  This is not correct, and causes trouble even for size 1 points if the
  derivative of the PointCoord value is ever used, or if the non-helper
  lane is not lane 0 (in which case the live lane may end up having
  PointCoord values outside [0,1].)

- SetupRoutine was using the clipspace position of the vertex. Xf,Yf
  are specified to be in screenspace. It's possible to do this in a
  less redundant way, but currently we don't have unrounded vertex
  positions in screenspace available in the setup program. Just
  reproject for now, as we do in clipping paths etc. Note that we only do
  this if we're drawing points, so most (filled triangle) draws should
  not incur any additional cost.

Bug: b/140680704
Test: dEQP-VK.glsl.builtin_var.simple.pointcoord*
Change-Id: If006cc92b86673b6bb747cf60f60e0ed46f5e8e3
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/36115Tested-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>

This adds to 'Bug:', 'Issue:' and 'Regres:' as accepted CL grep checks.

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

The scissor is now restricted to the framebuffer's size,
in order to avoid writing out of bounds of the image.

Vulkan 1.1 spec section 25.2. Scissor Test states that
"Rasterization does not produce fragments outside of the framebuffer"

Bug: b/141999739, angleproject:4060

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

MetalSurfaceEXT and MacOSSurfaceMVK were inheriting incorrectly from
the base class MetalSurface, and were improperly initialized.
Additionally, none of the drawables were ever released, meaning there
were no re-used drawables. Add an @autoreleasepool around the drawable
retrival path, as per the spec: https://developer.apple.com/documentation/quartzcore/cametallayer?language=objc#3172024

Bug: chromium:1015454
Change-Id: Idd2f1842977a6f5dccecf31d152d454a44e9261c
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/37948Reviewed-by: Alexis Hétu <sugoi@google.com>
Tested-by: Jonah Ryan-Davis <jonahr@google.com>

Change-Id: I81aa8a7fdfb1c900062cc68f030e8765e9f019a2
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/37528Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

VK_EXT_metal_surface is a similar extension to VK_MVK_macos_surface,
which is already supported.

Bug: chromium:1015454
Change-Id: I08d8e6b097d75236f724984b8c7d20251a63581f
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/37808
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Tested-by: Jonah Ryan-Davis <jonahr@google.com>

Copied from android/cts/master/vk-master.txt at dEQP revision
76a2b81e0ca86a65a927892ee4340bc6685e86b3 from
gitlab.khronos.org/Tracker/vk-gl-cts.git, which is the revision
currently used by Regres (see tests/regres/deqp.json).

WSI tests were removed from the Android vk-master.txt, as they only
contain Android WSI tests. Instead Regres uses vk-wsi.txt, which doesn't
need updating.

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

Bug: b/143479561
Change-Id: I25e68cc24f6141611493ea7dc24f90ab4b2e4d4b
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/37711Tested-by: Antonio Maiorano <amaiorano@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Bug: b/143479561
Change-Id: I555db2de99dfc5a139fbdf61c8ac4447e58ffd02
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/37709Reviewed-by: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>