Separate Operand store logic from EmitStore()

The new Store() helper function can store Operand instances independent
from SPIR-V instructions. This allows reuse of this logic for other
instructions that need to store to memory, like Modf and Frexp.

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

Only store component count in Operand

The Operand class is a low-level abstraction of rvalues and constants.
It should not carry the SPIR-V type ID. We only need the size in
components.

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

Rename size/sizeInComponents to componentCount

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

Use the type and result ID helpers

Avoid directly accessing SPIR-V instruction words. The helper methods
provide self-explanatory semantics so we can eliminate local variables
to store these IDs.

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

Obtain type ID from instruction

The defining instruction already contains the type ID.

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

Add helper methods for obtaining type and result ID

For instructions which define objects, the type ID and result ID are
always the second and third SPIR-V instruction word, respectively.

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

LLVMReactor: Replace deprecated CreateCall overload

This has been removed in LLVM master.

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

[cmake] Add support for CMAKE_SYSTEM_NAME == "Fuchsia"

This patch modifies the CMakeLists.txt file and a few sources
in order to build the SwiftShader Vulkan ICD for Fuchsia with
CMake.

Note that building EGL/GLES_CM/GLESv2 and tests is not supported,
and should be explicitly disabled when configuring the build,
e.g.:

  mkdir build-fuchsia && build-fuchsia
  cmake .. \
    -DCMAKE_TOOLCHAIN_FILE=.../FuchsiaSdkToolchain.cmake \
    -DCMAKE_BUILD_TYPE=Release \
    -DSWIFTSHADER_BUILD_TESTS=0 \
    -DSWIFTSHADER_BUILD_EGL=0 \
    -DSWIFTSHADER_BUILD_GLES_CM=0 \
    -DSWIFTSHADER_BUILD_GLESv2=0 \
    <other-cmake-options>

  make  # or ninja

NOTE: This should not affect the build for other platforms!

This will make it easier to conditionally add Fuchsia-specific
sources to the build (see discussion on b/144687651 for details).

The BUILD.gn file will still be maintained in parallel for
Fuchsia-specific changes.

NOTE: As of now, the Fuchsia SDK doesn't provide a way for
      third-party code to access the framebuffer directly.

      Instead, client code should rely on Vulkan presentation
      surfaces or higher-level APIs provided by the compositor
      that are still in flux / unstable. Hence this will appear
      in a future CL.

Bug: b/143122483
Bug: b/144687651
Change-Id: Ica3b98f5a8eb1370287e2bb4ff22ef938aaa4b49
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38488
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Tested-by: David Turner <digit@google.com>

Rename GenericValue to Operand

Bug: b/129000021
Change-Id: I0000fc5e65bde87e9037400002db37cb6d50960d
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43688
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>

CMake: Fix linux build with SWIFTSHADER_BUILD_PVR

The targets `libGLESv2` and `libEGL` are no longer public to the root cmake list.
Just move these copy ops into the subdirectory cmake rules without the PVR condition - the copy is cheap enough to do whether we're building PVR or not.

Bug: b/145758253
Change-Id: Ic28331e3797c78e4910332fce934c06c132c3aa8
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43710
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Vulkan/Debug: Fix clang-10 warning

It correctly points out that despite `Type() = default;`, there's non-assigned const fields, so there's no way to generate a default constructor.

Bug: b/145351270
Change-Id: I1c680982d960df278b5fcca5b29e837a4c82b70d
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43709
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

third_party/cppdap: Roll forward to 4dcca577

Changes:
    de7dffa Socket: Use SO_REUSEADDR, disable SO_LINGER
    9a9d46f Fix bad usage of std::move
    3e6cfd6 CMake: Make the cppdap target's include directory public
    4abe43c optional: Remove the value() method that returns non-const-ref.
    ced82a0 examples: Change stdin and stdout to binary mode
    96b25aa src/io.cpp: Fix uninitialized variable.
    93b8610 examples: Suffix package exe with .exe on Windows
    cdc19ac Serialization: Correctly encode structs with no fields
    3a10d4c Format all source files
    8633aba Add clang-format-all.sh to format all project sources
    1be9bb6 launch.json - fix name of unittests to launch
    d19d8f8 Address #7 nits and fix compiler warning (#9)
    73d697e Fix Response type info, make response 'body' field optional
    eab43f3 net::Server: Fix onError parameter default.

Bug: b/145351270
Change-Id: I6a7767f8b763e8029571d65aa5446f5043ad83ac
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43708
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Clamp _SNORM formats to -1.0

Following the Vulkan 1.1 specification Section 2.9.1. "Conversion from
Normalized Fixed-Point to Floating-Point":

Note that while zero is exactly expressible in this representation, one
value (-128 in the example) is outside the representable range, and must be
clamped before use.

https://www.khronos.org/registry/vulkan/specs/1.1/html/vkspec.html#fundamentals-fixedfpconv

Bug: dawn:283
Change-Id: Id548c6d67132ac36f33bc020954c5511de624ffc
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43648
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Corentin Wallez <cwallez@google.com>

Remove debug only code limiting the number of primitives

A test in dEQP was failing using SwANGLE in debug.
The test was using a 1200x1200 grid
 -> 1,44M squares
 -> 2,88M triangles

This was exceeding our hardcoded limit of 1 << 21 (2,097,152).

Removing this code makes the test pass.

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

Regres: Post coverage results even if the test lists have not changed.

Do this by breaking up the monolithic `runDaily()` function, and combining errors from the new `postDailyResults()` and `postCoverageResults()` functions.

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

CMake: split out all src/OpenGL targets into their own CMakeLists

Added CMakeLists.txt to the following folders that produce the
following targets:

src/OpenGL/common -> libGLESCommon (new)
src/OpenGL/compiler -> GLCompiler
src/OpenGL/libEGL -> libEGL
src/OpenGL/libGLESv2 ->libGLESv2
src/OpenGL/libGLES_CM -> libGLES_CM

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

CMake: split out SwiftShader GL deps into separate files

This change add a CMakeLists in each of the following folders, and
creates the following targets:

src/Common -> gl_common
src/Main -> gl_main
src/Renderer -> gl_renderer
src/Shader -> gl_shader

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

CMake: clean up dependencies

This change makes it so that source/Common and source/Main are only
referenced by the SwiftShader target. This will allow for better
target-based sub CMakeLists.

* Remove SharedLibrary.hpp/cpp from source list of libGLESv2 and
  libGLES_CM, as both of these depend on SwiftShader, which already
  builds these files.

* Make libEGL depend on SwiftShader, so we can remove
  SharedLibrary.hpp/cpp and Main/libX11.hpp/cpp from its
  source list.

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

Add SystemBenchmarks.

This benchmarks `sw::LRUCache`, which currently resides in the `src/Device` folder (but will be moved to `src/System`) with 43489.

Bug: b/153338950
Change-Id: I169b6ecb4a35349726a01b1da52472eaef3dfd74
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43492
Reviewed-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>

VkCommandBuffer: Remove static Cast() method

This isn't used.

There's another Cast() declared at vk scope, below VkCommandBuffer that actually is used.

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

Remove src/Device/Config.cpp

Holds sw::Profiler that is entirely dead code.
There's little here that looks worthy of keeping. Let's remove it.

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

Kokoro: Check that build files don't reference non-existent files

This is a primitive check to ensure that .bp, .gn and .bazel files don't have source references to missing files.

This can catch a common build breakage.

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

Regres: Fix uncovered calculations after optimizations

The uncovered span calculations were ignoring span groups and inverted spans.
This is now correctly handled by Tree.allSpans().

Bug: b/152192800
Change-Id: I2bd8afa3c956b03b598a3d5297cb775fd19da35d
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43573
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Regres: Optimize coverage processing

This had regressed mostly due to a single thread processing a huge number of `Span.Add()`s.

Optimize for the common case, avoid `Span.Add()` in places where the data is already sorted, don't scan the entire file system for every test run.

Bug: b/152192800
Change-Id: Id183468263e65bfbcf387ec1c978d8b9de547cee
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43570
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Regres: Re-populate treeFile.allSpans on parse.

This caused `TestTreeEncodeDecode` to fail, and would make the reparsing of a coverage file incorrect.

Bug: b/152192800
Change-Id: Ic9f4c49c58350509f74755fec20f22b6c1213877
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43569
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Update third_party/marl/BUILD.gn with new files

`include/marl/sal.h` was replaced with `include/marl/tsa.h`.
Also add other missing files.

Bug: b/140546382
Change-Id: I7f766b3c362de2ce6e4e317762de98db50467d8f
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43571
Reviewed-by: Alexis Hétu <sugoi@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Update Marl to 539094011

Includes new Thread Safety Analysis helpers

Changes:
    539094011 CMake: Export MARL_THREAD_SAFETY_ANALYSIS_SUPPORTED
    c7f70ba7a CMake: Bump min version + include(CheckCXXSourceCompiles)
    658a204fc Replace SAL annotations with clang's TSA annotations
    9630bec2f Fix CMake warning: "Policy CMP0023 is not set"
    9f369ad5d Update yarn:: to marl:: in an example

Commands:
    ./third_party/update-marl.sh --squash

Bug: b/140546382
Change-Id: Idb4253c11cece99ea4b22f965b974b63e26b51a7

Squashed 'third_party/marl/' changes from 16e1dc37c..539094011

539094011 CMake: Export MARL_THREAD_SAFETY_ANALYSIS_SUPPORTED
c7f70ba7a CMake: Bump min version + include(CheckCXXSourceCompiles)
658a204fc Replace SAL annotations with clang's TSA annotations
9630bec2f Fix CMake warning: "Policy CMP0023 is not set"
9f369ad5d Update yarn:: to marl:: in an example

git-subtree-dir: third_party/marl
git-subtree-split: 53909401165022553ed9d1f0c572178559dc25ec

Coverage: Improve uncovered visualizations

List all files, including those that have no coverage.
Emit spans that were compiled, but not covered.
Emit a % coverage per file.

Bug: b/152192800
Fixes: b/153182184
Change-Id: I31c831273a8d3ee89c7ce0737b6e05398ff4f51b
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43491
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Don't blend integer formats

True integer formats should not use blend modes.
This cl explicitly disables any blend operation
on render targets using integer formats.

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

Clamp GLES sine/cosine to [-1,1]

Hack for minor precision issues with current implementation. Fixes
8 quality warnings from deqp-gles3 when run against
SwiftShaderGL/Subzero.

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

Delete Plane.cpp and Plane.hpp

Cleaning up unused code

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

CMake / Kokoro: Build and run system-unittests.

These were added for the .gn build, but were never enabled for the CMake build.

Test these as part of the Kokoro presubmits.

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

Vulkan: Destruct the list of vk::Query in QueryPool

Bug: b/153168751
Bug: b/153206490
Change-Id: I2822f60aa380ae3885bc674d9f7b1f283c087e95
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43490
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Really fix Android build

Bug: none
Change-Id: I9f58cc3bf6b457fc8b916a98242138865f89c577
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43469
Reviewed-by: Ben Clayton <bclayton@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>

Fix Chromium and Android build

I broke this when I removed the relative path to the third_party astc
include file in this change:
https://swiftshader-review.googlesource.com/c/SwiftShader/+/43269

Bug: none
Change-Id: I45de8a6159c69c893df4449f6c6e10b09a9e6fe4
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43448
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>

Debug: Add missing include.

When `ENABLE_VK_DEBUGGER` is defined, VkDevice needs to be declared in order to build.

Bug: b/148401179
Change-Id: If35f3a1425ec0d739750d632305a625c7e12b721
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43432
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Vulkan/Debug: Create a PhysicalFile if DebugSource has no content

`VirtualFile`s are typically used for files with no physical backing.
Use `PhysicalFile` if there's no content provided in the debug info.

Bug: b/148401179
Change-Id: I1e1ed75ce9a9fb0965420267aba7f6d8de958012
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43430
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Vulkan/Debug: Always provide the dap::Source::path

This is common for virtual and physical files.

Bug: b/148401179
Change-Id: I266097382098c1daf776ae64d878f08febee5d0e
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43429
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

CMake: Don't always regenerate OpenCLDebugInfo100.h

https://github.com/KhronosGroup/SPIRV-Headers/issues/137 is still ongoing.

Fixes: b/15093697
Change-Id: I2498f35a8049fb2a1a86aa0efe6fbd6e59ad6493
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43428
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

CMake: use gtest and gmock targets

Bug: b/145758253
Bug: b/152606052
Change-Id: I826225ed393ed324301555ebaea14947997632cf
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43289
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>

CMake: split out atsc-encoder into its own CMakeLists

Also remove dependency of third_party/astc-encoder on
src/System/Debug.hpp.

Bug: b/145758253
Change-Id: I329928d9221c99a416fcdb1b0dd44d2578a22cb8
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43269
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

CMake: clean up cppdap usage

Remove unused and useless variables. Also fix the build when
SWIFTSHADER_ENABLE_VULKAN_DEBUGGER is enabled.

Bug: b/145758253
Change-Id: Ie49dd88d6716b24fe31baa8f8d162e7dacfa5a3c
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43268
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

CMake: split out Reactor into its own CMakeLists

Bug: b/145758253
Change-Id: I36f1bbf599389c27e084fea9af3d732225567076
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43230
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>

CMake: split out boost into its own CMakeLists

This adds a third_party/boost folder with a CMakeLists.txt. If this
folder is added via add_subdirectory, it downloads boost to the
binary directory and exposes the Boost::boost target.

Bug: b/145758253
Change-Id: I7d4e383630cb84da928cf0622a641e9e0efc2ad8
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43191
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>

CMake: split out libbacktrace into its own CMakeLists

This change also fixes compilation on Linux when REACTOR_EMIT_DEBUG_INFO
is enabled, which has been broken since "-fno-exceptions" was added to
SWIFTSHADER_COMPILE_OPTIONS. Also, at some point,
LIBBACKTRACE_CONFIG_DIR stopped being used, which further broke
compiling libbacktrace.

Bug: b/145758253
Change-Id: I50f0e2c2c1dabc357dfc255a471eaf1e9fed1c2e
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43189
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>

CMake: split out subzero and llvm-subzero into their own CMakeLists

Bug: b/145758253
Change-Id: Ife32c322cf87c9b1b98b4b50f591b75db6ea51e2
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43148
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Antonio Maiorano <amaiorano@google.com>

Kokoro: Add the ppa:ubuntu-toolchain-r/test apt-repository

Bug: None - Fix Kokoro presubmits.
Change-Id: I16074e733edd257647840d1bf51ebdc14dd4fb89
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43431
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Remove the D3D9 and D3D8 source code

This code has no remaining known active uses. It has been archived in
the 'legacy-d3d9' branch.

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

Remove the custom Visual Studio solution and projects

CMake should be used instead.

Note that the CMake build does not support the D3D9 and D3D8 targets.
They can still be built with Visual Studio 2019 from the 'legacy-d3d9'
branch.

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

Fix GN build

cf. https://swiftshader-review.googlesource.com/c/SwiftShader/+/43388

Bug: build fix
Change-Id: I8d2acc0bbb19fa71748aafeebd6c0d500673ed93
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43408
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>

Reactor: Fix remove_if brokenness

Spotted by Chris Forbes:
* `std::remove_if` wasn't introduced in C++20, it's been there since 98. I just can't read docs.
* Also fix the lack of `list.erase()`.

Bug: None. drive-by-code-reviewing-fixes.
Change-Id: I456d540ba973640747d6e4cb23faec65a4a9c83d
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43348
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Remove the sw::Color<> class

It was only still being used to store framebuffer blend constants.
Use sw::float4 instead.

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

Change for loops to use const refs, per performance-for-range-copy clang-tidy.

This CL optimizes C++11 range-based for loops where the variable is copied in each iteration but it would suffice to obtain it by const reference.  This is only applied to loop variables of types that are expensive to copy which means they are not trivially copyable or have a non-trivial copy constructor or destructor.

To ensure that it is safe to replace the copy with a const reference the following heuristic is employed:
  The loop variable is const qualified.
  The loop variable is not const, but only const methods or operators are invoked on it, or it is used as const reference or value argument in constructors or function calls.

See cl/304001946 for more information.

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

Fix ImageView size computation

Erroneous ImageView size computation was causing out of bounds
memory accesses whenever an ImageView only represents a subsection
of an Image. The getSizeInBytes() function now only returns the
size in bytes within the Image object from the beginning to the
end of the bytes used by the ImageView.

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

MetalSurface: set layer.device to a ref of the system default device.

If layer.device is null, [layer nextDrawable] returns nil, so on surface
initialization we set layer.device to a dummy MTLDevice by getting a
reference of the system default device.

MTLCreateSystemDefaultDevice() requires linking against
Metal.framework but Chromium needs to ship on platforms that don't have
Metal. This means we can't directly link against the framework, instead
we "weakly" link against it so that if it is not present, the function
pointers are just null (instead of failing to launch).

Bug: dawn:269
Change-Id: I5984613f576849c6f219c253da7eb277fa75ef52
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43209
Kokoro-Presubmit: Corentin Wallez <cwallez@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Corentin Wallez <cwallez@google.com>

Regres: Add invertCommon optimization.

invertCommon looks for tree nodes with the majority of the child nodes with the same spans.
This span is promoted up to the parent, and the children have the
span inverted.

This causes the span IDs to flip (inclusive / exlusive) each time they're seen while decending the tree. This adds a bit of complexity to the parsing of the data, but the file size reduction is significant.

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

Regres: Move optimization logic to separate file

There's more to come.

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

Regres: Keep coverage span groups stable.

Update tests.

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

Regres: Split coverage.go into multiple files

This was getting unruly.

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

Regres: Further optimizations for coverage

Compress common spans into groups and index these.
Reduces the full coverage json from ~500MB to just under 100MB.

Use zlib compression on the final file instead of a zip. The compression algorithm is the same, and produces roughly the same sized output, but a raw zlib file is much quicker to decompress with pako instead of jszip.

Bug: b/152192800
Change-Id: Id444ddf4027b6ace03570719f159ae14e481cd37
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43249
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Regres: Add cov.ReadJSON()

And add test to check it behaves as expected.

While currently nothing golang actually parses these files, this allows me to experiment with compression schemes on the full data set without needing to rebuild the coverage data from scratch each time.

Bug: b/152192800
Change-Id: I8070abe2bbebaef3e5889991060f1d1342078eb2
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43248
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Fix implicit narrowing cast

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

Eliminate the legacy PowerVR SDK

We now have an up-to-date third_party/PowerVR_Examples submodule for
these sample applications.

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

Integrate PowerVR OpenGL ES samples

Also add some additional Vulkan samples to the 'good' set.

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

Fix completeness test of OpenGL ES immutable textures

Immutable textures created with glTexStorage*() are always mipmap and
sampler complete. The spec states that

"Array levels k where k < levelbase or k > q arebinsignificant to the
definition ofcompleteness."

Where q is no larger than levelmax, which is in turn limited to
levelimmut - 1 which was specified at glTexStorage.

Change-Id: Ieaa6be856fd35cf2827e753d7c777ebf784027ef
Tests: dEQP-GLES*
Fixes: b/152740217
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43188
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

Revert "MetalSurface: set layer.device to a ref of the system default device."

This reverts commit 81d8c2ada561340400ccda0c560dc8f61ffb9a30.

Reason for revert: Breaks macOS swiftshader build

Change-Id: I1c54aaf772f651d731d90e843c85549a4f7a38b8
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43208
Tested-by: Corentin Wallez <cwallez@google.com>
Reviewed-by: Corentin Wallez <cwallez@google.com>

MetalSurface: set layer.device to a ref of the system default device.

If layer.device is null, [layer nextDrawable] returns nil, so on surface
initialization we set layer.device to a dummy MTLDevice by getting a
reference of the system default device.

Using MTLCreateSystemDefaultDevice() requires linking against
Metal.framework but Chromium needs to ship on platforms that don't have
Metal. This means we can't directly link against the framework, instead
we "weakly" link against it so that if it is not present, the function
pointers are just null (instead of failing to launch).

Bug: dawn:269
Change-Id: I8719c45a19718ff79ef21f47515fe1c15b99628b
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43112
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Corentin Wallez <cwallez@google.com>

Don't expose the sampling routine cache's mutex

The getOrCreate() method takes care of the mutex locking and unlocking,
and takes the function for creating the routine as a callback argument.

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

Create PowerVR targets if submodule already exists

If the PowerVR_Examples submodule has been checked out, enable creating
the CMake targets for them. This helps make them available in all
configurations in Visual Studio, without editing the cache for each of
them individually.

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

Regres: Generate coverage data on nightly runs

Push this to https://github.com/swiftshader-regres/swiftshader-coverage so that it can be viewed at:
https://swiftshader-regres.github.io/swiftshader-coverage/

There's quite a lot of new code and fixes in this change. The most notable:
* The regres daily run for the subzero backend now produces combined coverage information for all the test runs. The LLVM backend does not produce coverage information.
* Regres now takes two additional command line arguments: `gh-user` and `gh-pass` for the swiftshader-regres account. If you omit these, then coverage will not be produced.
* test.srcDir has been renamed to checkoutDir, as this was confusing with the `src` directory in the repo.
* The coverage JSON now contains a root field to describe the git revision to which it relates. This prevents the coverage going out of sync with the source.
* `git.CheckoutRemoteBranch()` drops back to a depth of 1 again. This was only increased to 99 to deal with issues checking out from gitlab, which we don't do any more.
* Regres  now builds using `third_party/llvm-10.0`
* Fixed the `--limit` regres command line flag which wasn't actually limiting, as it was using the len() on the number of groups, not the number of tests.

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

Subzero Coroutines: Use ultra-low-level marl APIs

Instead of using the `marl::Event` synchronization primitives, drop all the way down to using `marl::Scheduler::Fiber` APIs directly, and use the new `marl::Scheduler::Fiber::wait()` function that does not take a lock.

Provides around another 2x performance boost:

```
go run ./third_party/marl/tools/cmd/benchdiff/main.go pre.json post.json
Delta                | Test name                                | (A) pre.json | (B) post.json
-2.08x -81.186µs     | Coroutines/Fibonacci/iterations:512      | 156.202µs    | 75.016µs
-2.10x -668.201µs    | Coroutines/Fibonacci/iterations:4096     | 1.275069ms   | 606.868µs
-2.11x -5.359326ms   | Coroutines/Fibonacci/iterations:32768    | 10.166126ms  | 4.8068ms
-2.12x -10.342µs     | Coroutines/Fibonacci/iterations:64       | 19.585µs     | 9.243µs
-2.13x -2.784542745s | Coroutines/Fibonacci/iterations:16777216 | 5.251299045s | 2.4667563s
-2.13x -347.220746ms | Coroutines/Fibonacci/iterations:2097152  | 653.812928ms | 306.592182ms
-2.14x -43.615678ms  | Coroutines/Fibonacci/iterations:262144   | 82.017312ms  | 38.401634ms
```

Bug: b/145754674
Change-Id: I5f6b0c8c92af645cc2a825c6f1e2769b2440638e
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/42850
Tested-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Subzero: Ensure coroutines are same-thread.

Marl now has a new flag on the task to say that it should be run on the same thread that calls `marl::schedule()`.

This can dramatically reduce the scheduling overhead when bouncing between two tasks, as other threads do not have to be woken.

```
> go run ./third_party/marl/tools/cmd/benchdiff/main.go pre.json post.json
Delta                    | Test name                                | (A) pre.json    | (B) post.json
-15.53x -1m16.299750679s | Coroutines/Fibonacci/iterations:16777216 | 1m21.551049724s | 5.251299045s
-15.71x -1.206771937s    | Coroutines/Fibonacci/iterations:262144   | 1.288789249s    | 82.017312ms
-15.72x -9.624071378s    | Coroutines/Fibonacci/iterations:2097152  | 10.277884306s   | 653.812928ms
-15.77x -35.755µs        | Coroutines/Fibonacci/iterations:8        | 38.176µs        | 2.421µs
-15.77x -150.190662ms    | Coroutines/Fibonacci/iterations:32768    | 160.356788ms    | 10.166126ms
-15.79x -18.864275ms     | Coroutines/Fibonacci/iterations:4096     | 20.139344ms     | 1.275069ms
-15.93x -2.332202ms      | Coroutines/Fibonacci/iterations:512      | 2.488404ms      | 156.202µs
-15.96x -292.896µs       | Coroutines/Fibonacci/iterations:64       | 312.481µs       | 19.585µs
```

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

Rename constant cache to snapshot cache

This helps clarify that the cache is able to take snapshots of its
contents, and this snapshot can be queried without the overhead of
acquiring a mutex.

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

Add sampling routine documentation

Bug: b/152424696
Change-Id: Ib064ff7f6d88bf965e58c0b640cd90f7f1bc2d83
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/42988
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Sean Risser <srisser@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Fix Android.bp following the SPIRV-Tools update

Following the SPIRV-Tools update, Android.bp needs to be manually
updated to reflect the changes made in SPIRV-Tools.

This change was tested with SwANGLE on Android.

Bug: b/123642959
Change-Id: I3ba86796a0a5e75d642ef13a0f06735035f1bd5c
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43128
Tested-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>

Regres: Switch to building with LLVM 10

This is automatically downloaded and cached.
This is another step towards having hermetic builds, and is required for generating code coverage.

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

Regres: Add llvm.Version.Download() to download and verify a LLVM toolchain

Will be used to have regres switch to LLVM 10.

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

Kokoro: Test LLVM 10 backend for ubuntu

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

Coverage: Add 'turbo-cov': a faster reimplementation of llvm-cov

`llvm-cov` can either emit data in json or lcov formats, where json is the faster of the two.
`llvm-cov`'s output is directly piped into regres, where the json is immediately deserialized again.
The cost of serializing and deserializing is surprisingly high.

This change replaces the use of `llvm-cov` with `turbo-cov`, which simply emits a binary stream and offers up to 3x speed improvement, dramatically lowering the time taken to produce coverage for a full deqp test run.

Bug: b/152339534
Change-Id: I9292f3c27e016cf508557edf4da7656db81c2b07
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/42948
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Don't download all submodules for Kokoro tests

The necessary submodules should be downloaded automatically.

Bug: b/141361933
Change-Id: Icd4690dccb851ecdc67a8f216a5727f1d8b7142a
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43068
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>

Reactor: Disable more warnings to get LLVM 10 building

There's yet more unnecessary semicolons knocking around in the public LLVM headers, which are purely stylistic issues. Just silence them.

`LegacyIRCompileLayer` is now annotated with deprecated, so we need to disable the warning for that too (and move to the new JIT in time).

Bug: b/152339534
Change-Id: Id84fe88fd2f6b50000e37a8b84f52742ca9db646
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43010
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

LLVM 10: Add configs/common and configs/darwin.

Configs still to add: android, fuchsia, windows

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

LLVM 10: Add configs/common and configs/linux.

Configs still to add: android, darwin, fuchsia, windows

Bug: b/152339534
Change-Id: I721b7a69a0a1e84f09158329557264bf2af9c5d6
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43009
Tested-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Update SPIR-V Tools to fd773eb50

Includes fix #3253, which fixed the following SwANGLE test:
KHR-GLES31.core.compute_shader.resources-max

Changes:
    fd773eb50 Start SPIRV-Tools v2020.3
    fd8e13051 Finalize SPIRV-Tools v2020.2
    3ef8fe9a5 Update CHANGES
    022da4d0e Fix identification of Vulkan images and buffers (#3253)
    1346dd5de Disallow phis of images, samplers and sampled images (#3246)
    1c8bda372 Add data structure for DebugScope, DebugDeclare in spirv-opt (#3183)
    e6f372c5c Whitelist SPV_KHR_ray_tracing (#3241)
    7f341ffee Make file formatting comply with POSIX standards (#3242)
    60104cd97 Add opt::Operand::AsCString and AsString (#3240)
    da52d0875 Add RayQueryProvisionalKHR to opt types (#3239)
    18d3896a1 Whitelist SPV_EXT_demote_to_helper_invocation for opt passes (#3236)
    5a97e3a39 Add support for KHR_ray_{query,tracing} extensions (#3235)
    25ede1ced Roll external/spirv-headers/ 30ef660ce..a17e17e36 (1 commit) (#3230)
    7a8f79762 Update dependencies (#3228)
    1fe9bcc10 Instrument: Debug Printf support (#3215)
    6428ad05e spirv-fuzz: Support OpPhi when adding dead break and continue (#3225)
    1af1df3b2 spirv-fuzz: Fix vector width issue in 'add equation instructions' pass (#3223)
    dd3d91691 Allow sampledimage types as operand of OpCopyObject (#3222)
    7c3de218f spirv-fuzz: Remove duplicated functionality (#3220)
    659470446 spirv-fuzz: Allow OpPhi operand to be replaced with a composite synonym (#3221)
    4c027048d spirv-fuzz: Add toggle access chain instruction transformation (#3211)
    533af4981 spirv-fuzz: Add fuzzer pass to permute function parameters (#3212)
    da4cd2148 spirv-fuzz: Use better function name (#3207)
    66a682b6a spirv-fuzz: Add swap commutable operands transformation (#3205)
    044ecc0b2 spirv-fuzz: Fuzzer pass to add equation instructions (#3202)
    a6d3a2dd4 Refactor FuzzerPass::ApplyTransformation code duplication. (#3206)
    e1688b60c Avoid use of Python distutils.dir_util (#3203)
    661e79eec Adding WebGPU specific Workgroup scope rule (#3204)
    70f888131 Add validation rules for OpenCL.DebugInfo.100 extension (#3133)
    fb6e3e48d Combine extinst-name and extinst-output-base into one arg. (#3200)

Commands:
    ./third_party/update-spirvtools.sh

Bug: b/123642959
Bug: b/148460089
Change-Id: I8fa5d3609de1c6ae786c84b93cba7ac015b4f56e

Squashed 'third_party/SPIRV-Tools/' changes from 8910ea5f1..fd773eb50

fd773eb50 Start SPIRV-Tools v2020.3
fd8e13051 Finalize SPIRV-Tools v2020.2
3ef8fe9a5 Update CHANGES
022da4d0e Fix identification of Vulkan images and buffers (#3253)
1346dd5de Disallow phis of images, samplers and sampled images (#3246)
1c8bda372 Add data structure for DebugScope, DebugDeclare in spirv-opt (#3183)
e6f372c5c Whitelist SPV_KHR_ray_tracing (#3241)
7f341ffee Make file formatting comply with POSIX standards (#3242)
60104cd97 Add opt::Operand::AsCString and AsString (#3240)
da52d0875 Add RayQueryProvisionalKHR to opt types (#3239)
18d3896a1 Whitelist SPV_EXT_demote_to_helper_invocation for opt passes (#3236)
5a97e3a39 Add support for KHR_ray_{query,tracing} extensions (#3235)
25ede1ced Roll external/spirv-headers/ 30ef660ce..a17e17e36 (1 commit) (#3230)
7a8f79762 Update dependencies (#3228)
1fe9bcc10 Instrument: Debug Printf support (#3215)
6428ad05e spirv-fuzz: Support OpPhi when adding dead break and continue (#3225)
1af1df3b2 spirv-fuzz: Fix vector width issue in 'add equation instructions' pass (#3223)
dd3d91691 Allow sampledimage types as operand of OpCopyObject (#3222)
7c3de218f spirv-fuzz: Remove duplicated functionality (#3220)
659470446 spirv-fuzz: Allow OpPhi operand to be replaced with a composite synonym (#3221)
4c027048d spirv-fuzz: Add toggle access chain instruction transformation (#3211)
533af4981 spirv-fuzz: Add fuzzer pass to permute function parameters (#3212)
da4cd2148 spirv-fuzz: Use better function name (#3207)
66a682b6a spirv-fuzz: Add swap commutable operands transformation (#3205)
044ecc0b2 spirv-fuzz: Fuzzer pass to add equation instructions (#3202)
a6d3a2dd4 Refactor FuzzerPass::ApplyTransformation code duplication. (#3206)
e1688b60c Avoid use of Python distutils.dir_util (#3203)
661e79eec Adding WebGPU specific Workgroup scope rule (#3204)
70f888131 Add validation rules for OpenCL.DebugInfo.100 extension (#3133)
fb6e3e48d Combine extinst-name and extinst-output-base into one arg. (#3200)

git-subtree-dir: third_party/SPIRV-Tools
git-subtree-split: fd773eb50d628c1981338addc093df879757c2cf

Update SPIR-V Headers to f8bf11a02

Changes:
    f8bf11a02 Merge pull request #149 from dgkoch/prov_ray_tracing
    fdbc0d1c4 Add shadercalls scope
    9a186c772 Added ray flags, primitive culling flags, queries
    9e8e6aff3 Non-functional: Update header build to match Khronos spec. builder.
    8830d1d77 Merge pull request #135 from vulturm/patch-1
    95b48cedd Update headers for SPV_KHR_ray_tracing.
    a17e17e36 Merge pull request #148 from null77/fix-gn
    976926d58 Add missing header to BUILD.gn.
    30ef660ce Merge pull request #146 from s-perron/bazel
    0d334e4ad Export NonSemanticDebugPrintf.h in bazel build
    d9ce9ca52 Merge pull request #145 from jeffbolznv/nonsemantic_debugprintf
    e814bf006 Add NonSemantic.DebugPrintf JSON/header
    89bef4096 Fix max enum value
    0a7fc4525 Add grammars, C header, and header generator for vendor and KHR extended instruction sets (#143)
    5dbc1c321 Merge pull request #142 from mkinsner/additional_loop_control_bits
    4b013f0fd Allocate three bits for upcoming Intel extension
    dc77030ac Merge pull request #141 from dneto0/update-buildgn-lic
    9c63d5773 Fix the license to match LICENSE
    0125ae61b Merge pull request #140 from ShabbyX/add_build_gn
    c7f52d34c Add BUILD.gn
    927aae4dc Also propagate SPIRV-Headers version to CMakeLists.txt

Commands:
    ./third_party/update-spirvheaders.sh

Bug: b/123642959
Change-Id: Iaacec0b1b56e49b53a16e27ddea851ee02cf6d3f

Squashed 'third_party/SPIRV-Headers/' changes from 204cd131c..f8bf11a02

f8bf11a02 Merge pull request #149 from dgkoch/prov_ray_tracing
fdbc0d1c4 Add shadercalls scope
9a186c772 Added ray flags, primitive culling flags, queries
9e8e6aff3 Non-functional: Update header build to match Khronos spec. builder.
8830d1d77 Merge pull request #135 from vulturm/patch-1
95b48cedd Update headers for SPV_KHR_ray_tracing.
a17e17e36 Merge pull request #148 from null77/fix-gn
976926d58 Add missing header to BUILD.gn.
30ef660ce Merge pull request #146 from s-perron/bazel
0d334e4ad Export NonSemanticDebugPrintf.h in bazel build
d9ce9ca52 Merge pull request #145 from jeffbolznv/nonsemantic_debugprintf
e814bf006 Add NonSemantic.DebugPrintf JSON/header
89bef4096 Fix max enum value
0a7fc4525 Add grammars, C header, and header generator for vendor and KHR extended instruction sets (#143)
5dbc1c321 Merge pull request #142 from mkinsner/additional_loop_control_bits
4b013f0fd Allocate three bits for upcoming Intel extension
dc77030ac Merge pull request #141 from dneto0/update-buildgn-lic
9c63d5773 Fix the license to match LICENSE
0125ae61b Merge pull request #140 from ShabbyX/add_build_gn
c7f52d34c Add BUILD.gn
927aae4dc Also propagate SPIRV-Headers version to CMakeLists.txt

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

Remove two minor compiler warnings/errors:

Found when building with the Fuchsia toolchain, which
uses a recent clang version:

- The constants '1.0f / 0x7FFFFFFF' and
  '1.0f / 0xFFFFFFFF' cannot be computed directly without
  losing one bit of accuracy, and the compiler was complaining
  about it, so use intermediate double values
  to get the correct, final result.

- The MemoryMapped class needs to be declared final to
  have a final destructor, otherwise the compiler complains
  with:

    error: class with destructor marked 'final' cannot be inherited from
    [-Werror,-Wfinal-dtor-non-final-class]

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

third_party: Create scripts for updating spirv-[tools|headers]

Bug: None
Change-Id: I8a2a39894f022d93ad159fbfd20e34fbc06fa73d
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43050
Reviewed-by: Alexis Hétu <sugoi@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Fix taking aspect into account for image view identifiers

Tests like dEQP.GLES3/functional_fbo_blit_depth_stencil_depth24_stencil8_basic
were not passing when running on SWANGLE.

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

Kokoro: Move llvm configs into 7.0 subdirectory, create 10.0 copies.

And add the LLVM_VERSION env var and plumb this into the shell scripts.
Stub building the 10.0 builds for now. We'll enable them one at a time.

Bug: b/152339534
Change-Id: I645a6d748dec956505ba550ce96ff85a21f9f73e
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43012
Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>

Regres: Add support for generating collated coverage info

`run_testlist` now supports the `--coverage` flag, which will collate the coverage information from each and every deqp test that's run.
This information is written to a `coverage.json` file which can be consumed by a html browser.

An early version of this can be found here (likely to move):
https://ben-clayton.github.io/swiftshader-coverage/

Bug: b/152192800
Change-Id: I52434f1ce30e6a091d2932fbae309cd81809cb79
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/42890
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

CMake: Emit coverage-toolchain.txt file next to ICD json

When `SWIFTSHADER_EMIT_COVERAGE` is enabled, generate a new `coverage-toolchain.txt` text file next to the `vk_swiftshader_icd.json` file. This file contains a path to the C++ compiler toolchain used to build swiftshader, so that the regres tooling can locate the llvm tools used to parse the coverage data.

Bug: b/152192800
Change-Id: I3aa718206c3262bcd05e13e854002dea8f40801f
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/42889
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

CMake: Add SWIFTSHADER_EMIT_COVERAGE option

Enables code coverage generation for gcc and clang toolchains.

Bug: b/152192800
Change-Id: If4cca74f2c5edf23626fe6053f29928e62ec00d3
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/42709
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

CMake: Add option for selecting LLVM version.

And move LLVM_DIR and LLVM_CONFIG_DIR into the llvm's CMakeLists.txt

Bug: b/145758253
Bug: b/152339534
Change-Id: I29e555e09b82dcf77cd9e9d57b5fb8a477473bbe
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43008
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>

Merge "Initial drop of LLVM 10 to third_party/llvm-10.0"

Identify image views based on their state

It is typical for apps to have many image views (and buffer views) with
the same parameters. This change avoids generating new sampling routines
for image views with identical state.

The vk::Identifier class compresses the state into a 32-bit value that
is used as part of the key for the sampling routine cache.

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

Provide packed VkFormat to uint8_t mapping

There are only 241 unique format enums at this moment, so they can fit
in an 8-bit variable instead of the 32-bit enum.

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

Uniquely identify sampler state

To avoid re-generating sampling routines for sampler with identical
state, keep a map of sampler state to 32-bit integer identifiers.

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