- 06 Jan, 2020 7 commits
-
-
Ben Clayton authored
* changes: Update SPIRV-Tools to 8aa423930 Squashed 'third_party/SPIRV-Tools/' changes from c3f22f7cb..8aa423930 VulkanUnitTests: Fix SPIR-V validation issue Update SPIRV-Headers to 204cd131c Squashed 'third_party/SPIRV-Headers/' changes from af64a9e82..204cd131c
-
Alexis Hetu authored
Making sure we extend 1010102 to the full 16 bit range fixes 51 of the 53 failures found in ToT dEQP-VK.*a2b10* Also added a utility function to OR all elements of an int vector, which allows us to use more vector operations (as opposed to scalar) while improving the readability. Bug: b/146633956 Change-Id: If8b946c45cf27f5868d7a97166e21dba565ed72f Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39768Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Contains initial support for OpenCL.DebugInfo.100 extended instruction set Changes: 8aa423930 Avoid pessimizing std::move (#3124) fad92f1e7 Fix typo in validation message (#3122) 578c5ac13 Change default version for CreatInstBindlessCheckPass to 2 (#3119) 9215c1b7d Fix convert-relax-to-half invalid code (#3099) (#3106) 64f36ea52 Support OpenCL.DebugInfo.100 extended instruction set (#3080) e01bc6d4e spirv-fuzz: Always add new globals to entry point interfaces (#3113) dcb7169bb spirv-fuzz: Transformation to add a new function to a module (#3114) 2e41d5ece spirv-fuzz: Avoid passing access chains as parameters (#3112) e70b009b0 Add support for SPV_KHR_non_semantic_info (#3110) 38d7fbaad spirv-fuzz: Transformations to add types, constants and variables (#3101) fccbc00ac Make Instrumentation format version 2 the default (Step 1) (#3096) 96354f504 spirv-fuzz: Fuzzer pass to merge blocks (#3097) 5c019b592 Start SPIRV-Tools v2020.1 c413b982c Finalize SPIRV-Tools v2019.5 2afbe9051 Update CHANGES 00ca4e5bd Don't crash when folding construct of empty struct (#3092) 0a2b38d08 spirv-fuzz: function outlining fuzzer pass (#3078) 983b5b4fc spirv-fuzz: Use validator to check break/continue dominance conditions (#3089) e82a42860 WebGPU: Array size at most max signed int + 1 (#3077) 0a5d99d02 Permit the debug instructions in WebGPU SPIR-V - remove from the optimizer (#3083) af7410597 graphics robust access: use signed clamp (#3073) 3ed458604 Folding: perform add and sub on mismatched integer types (#3084) 47f3eb426 spirv-fuzz: Fix invalid tests (#3079) b334829a9 Validate nested constructs (#3068) 52e9cc930 spirv-fuzz: Improve debugging facilities (#3074) 54385458c Handle unreachable block when computing register pressure (#3070) 868ca3954 Improve RegisterSizePasses (#3059) f31f26f73 utils/vscode: Add install.bat (#3071) 03957e8a9 build: cmake: Add support for Fuchsia. (#3062) a62012ced Add test with explicit example of stripping reflection info (#3064) 8312c523e Permit the debug instructions in WebGPU SPIR-V (#3063) 85f3e93d1 Respect CMAKE_INSTALL_LIBDIR in installed CMake files (#3054) 45dde9ad6 Add missing dealloc (#3061) 2ee9aaa28 Initialize binary for use as guard later (#3058) 0391d0823 Handle OpPhi with no in operands in value numbering (#3056) ca703c887 Kill the id-to-func map after wrap-opkill (#3055) 57b4cb40b Convert stderr and stdout in status to strings on assignment (#3049) Commands: git subtree pull --prefix third_party/SPIRV-Tools https://github.com/KhronosGroup/SPIRV-Tools master --squash Bug: b/123642959 Change-Id: Id1120acb6126f62c3928774c985fcc09d2619584
-
Ben Clayton authored
8aa423930 Avoid pessimizing std::move (#3124) fad92f1e7 Fix typo in validation message (#3122) 578c5ac13 Change default version for CreatInstBindlessCheckPass to 2 (#3119) 9215c1b7d Fix convert-relax-to-half invalid code (#3099) (#3106) 64f36ea52 Support OpenCL.DebugInfo.100 extended instruction set (#3080) e01bc6d4e spirv-fuzz: Always add new globals to entry point interfaces (#3113) dcb7169bb spirv-fuzz: Transformation to add a new function to a module (#3114) 2e41d5ece spirv-fuzz: Avoid passing access chains as parameters (#3112) e70b009b0 Add support for SPV_KHR_non_semantic_info (#3110) 38d7fbaad spirv-fuzz: Transformations to add types, constants and variables (#3101) fccbc00ac Make Instrumentation format version 2 the default (Step 1) (#3096) 96354f504 spirv-fuzz: Fuzzer pass to merge blocks (#3097) 5c019b592 Start SPIRV-Tools v2020.1 c413b982c Finalize SPIRV-Tools v2019.5 2afbe9051 Update CHANGES 00ca4e5bd Don't crash when folding construct of empty struct (#3092) 0a2b38d08 spirv-fuzz: function outlining fuzzer pass (#3078) 983b5b4fc spirv-fuzz: Use validator to check break/continue dominance conditions (#3089) e82a42860 WebGPU: Array size at most max signed int + 1 (#3077) 0a5d99d02 Permit the debug instructions in WebGPU SPIR-V - remove from the optimizer (#3083) af7410597 graphics robust access: use signed clamp (#3073) 3ed458604 Folding: perform add and sub on mismatched integer types (#3084) 47f3eb426 spirv-fuzz: Fix invalid tests (#3079) b334829a9 Validate nested constructs (#3068) 52e9cc930 spirv-fuzz: Improve debugging facilities (#3074) 54385458c Handle unreachable block when computing register pressure (#3070) 868ca3954 Improve RegisterSizePasses (#3059) f31f26f73 utils/vscode: Add install.bat (#3071) 03957e8a9 build: cmake: Add support for Fuchsia. (#3062) a62012ced Add test with explicit example of stripping reflection info (#3064) 8312c523e Permit the debug instructions in WebGPU SPIR-V (#3063) 85f3e93d1 Respect CMAKE_INSTALL_LIBDIR in installed CMake files (#3054) 45dde9ad6 Add missing dealloc (#3061) 2ee9aaa28 Initialize binary for use as guard later (#3058) 0391d0823 Handle OpPhi with no in operands in value numbering (#3056) ca703c887 Kill the id-to-func map after wrap-opkill (#3055) 57b4cb40b Convert stderr and stdout in status to strings on assignment (#3049) git-subtree-dir: third_party/SPIRV-Tools git-subtree-split: 8aa423930db37e37086665efcc55944d577c06e5
-
Ben Clayton authored
`LoopDivergentMergePhi` was attempting to use the same merge block for a `OpLoopMerge` and a `OpSelectionMerge`. This is not legal, and was caught by improvements to the SPIR-V tools validator. The fix is simple: Add a new merge block for the inner `OpSelectionMerge` that immediately jumps to the merge block for the `OpLoopMerge`. Bug: b/123642959 Bug: b/133440380 Bug: b/133481698 Change-Id: I8b5da04a1cd2db3e44bb5b50f1619a8565e5bc9d
-
Ben Clayton authored
Changes: 204cd131c Merge pull request #134 from Tobski/patch-1 e0adad52b Off-by-one errors 6045738ac Reserve a new block of 64 opcodes Commands: git subtree pull --prefix third_party/SPIRV-Headers https://github.com/KhronosGroup/SPIRV-Headers master --squash Bug: b/123642959 Change-Id: I9389ad9ea7916464745c2dfedefe46e3ef3a52e4
-
Ben Clayton authored
204cd131c Merge pull request #134 from Tobski/patch-1 e0adad52b Off-by-one errors 6045738ac Reserve a new block of 64 opcodes git-subtree-dir: third_party/SPIRV-Headers git-subtree-split: 204cd131c42b90d129073719f2766293ce35c081
-
- 30 Dec, 2019 1 commit
-
-
Chris Forbes authored
While debugging the 1.2.1 dEQP-VK.graphicsfuzz.* tests, noticed a crash here where a non-constant failed to have its intermediate pointer set. An assert here is clearer than exploding. Bug: b/146653181 Change-Id: I23aae701edce1595cd145930d304c9f6e4b82924 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39808Tested-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>
-
- 27 Dec, 2019 1 commit
-
-
Nicolas Capens authored
It was already unused. Bug: b/129403963 Change-Id: Idd4d7d1b64bb6380364e86823787ce54bc1cf8ab Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39730 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-
- 20 Dec, 2019 4 commits
-
-
Nicolas Capens authored
Surface::size() adds 4 padding bytes due to reading byte4 as 8 bytes for efficient unpacking to short4 (less efficient prior to SSE 4.1 pmovzxbw instruction). This padding is never addressed separately so the offset can't overflow a signed 32-bit integer if we keep the total size minus the padding below 2 GiB. On the OpenGL side we currently impose a further limit of 1 GiB, but this should also take the padding into account to allow for effective image data up to this limit (e.g. 8192 * 8192 of four samples of 32-bit elements). Note that our scanline limit is also 8192 so allowing more than 1 GiB would run into that limit instead. Bug: chromium:835299 Bug: b/145229887 Change-Id: I4280fca88584235c0da6282ca622ee15b31bc34b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39690 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>
-
Antonio Maiorano authored
* Add rr::BackendName() to dynamically return the name of the backend * Set VkPhysicalDeviceProperties::deviceName to include backend name Bug: b/130459196 Change-Id: I028f3607e4eda9de55bc4cec8af78e7b4793160d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39552 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Antonio Maiorano <amaiorano@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Antonio Maiorano authored
If enabled, TRACE macros (in Reactor and Vulkan) will use OutputDebugString on Windows, printf otherwise, to print the trace. On Windows, this allows us to use DbgView to monitor and filter the trace easily, especially useful when running with Chrome, for e.g. Bug: b/146051794 Change-Id: I15a46f5f0e8bc4e9edcacaf63c15dc605323356d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39568 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Antonio Maiorano <amaiorano@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I092acdfdd47e91aca0530abba035e8b575fcf0db Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39788 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
- 19 Dec, 2019 3 commits
-
-
Alexis Hetu authored
Exposing GL_OES_depth_texture requires that at least depth only format are filterable, so making sure at least D16_UNORM and D32_SFLOAT can be filtered. Tests: dEQP-VK.*d16_unorm* Tests: dEQP-VK.*d32_sfloat* Bug: b/146563038 Change-Id: I2c5a10b03eab19d179c005e2f0f8f22a4921830c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39748 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Checks for Bug: line, licenses, clang-format (v 8.0.0) and gofmt. These checks are performed by a separate kokoro instance. Fixed up broken licenses that it found. Counterpart: cl/286343852 Bug: b/144825072 Bug: b/141892461 Change-Id: If4ffda0b6144efe3cd5560cf17a6f4ad33bf985a Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39708Reviewed-by:
Antonio Maiorano <amaiorano@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I923f535fdec3a139e5d4d9be204ecd5a6b99a4b3 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39709Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
- 18 Dec, 2019 6 commits
-
-
Chris Forbes authored
The fix for graphicsfuzz tests having invalid control flow is now landed, so we no longer need to carry the patch. Bug: b/144183516 Change-Id: Ic65a853ddf90b085068874f0c50ab0793f109f8e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39728Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Alexis Hetu authored
In order for SwANGLE to expose GL_OES_texture_float_linear, VK_FORMAT_R.*32_SFLOAT formats must expose the VK_FORMAT_FEATURE_SAMPLED_IMAGE_FILTER_LINEAR_BIT bit. Tests: dEQP-VK.pipeline.sampler*32_sfloat*filter.linear Bug: b/146488564 Change-Id: I0b3d2832a76488f778a1978f6e33fb9bdb5ca05a Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39689Tested-by:
Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
These are unused by the Vulkan implementation. We have sw::float4 instead for vector data type needs. Bug: b/146224130 Change-Id: Icbb01e3ea82b760ad60f99c10e65c38e825ef12d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39688 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> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
SwiftShader Regression Bot authored
Change-Id: Ic0491111099041909c723e622fd2ef89f55b860e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39660Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
- Enabled the feature - Removed restriction from SpirvShader::getImageSampler() - Added support for it in SpirvShader::convertFilterMode() The rest of the logic to compute anisotropic filtering was still there. Tests: dEQP-VK.texture.filtering_anisotropy.* Bug: b/146438763 Change-Id: I2a36d00b6da88091202abe6d4f4f7f9b49507eb3 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39668 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Chris Forbes authored
We use the sample count from the pipeline in all cases, so we don't have to do anything special for this. Bug: b/146387550 Change-Id: I1f1fd33a257bcb9d528efef94d0c919a42d56400 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39608Tested-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 17 Dec, 2019 15 commits
-
-
Ben Clayton authored
Bug: b/144825072 Change-Id: Iabb4e1b2f4bb3da842fdeaa3ad2c6c3816810ae6 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39659 Presubmit-Ready: Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/144825072 Change-Id: I1bd5196b34a7974a41dcb95814a1ae8643b26f22 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39658 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/144825072 Change-Id: Ifc469786ac5fa2a8ed00e95069e1a2839e320e96 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39657 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/144825072 Change-Id: I1f14af4446536c760d11d32aa03edb5f497e75e4 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39656 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/144825072 Change-Id: I869aef91d6318bf6955581e5dad762800bd46296 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39655Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/144825072 Change-Id: I2e9c18aa5e3d394d18f86ca597aed6d6dc8dfe93 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39654 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Also add clang-format-all.sh and clang-format-separate.sh to perform this operation. These files sit in the ./src directory. Bug: b/144825072 Change-Id: I5228ac59f7781ad1307c305949af7e7f9e0c31d1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/24308Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Nicolas Capens authored
Spaces are useful to separate independent constructs, but can cause confusion when added between dependent ones. For example "a*b [i]" is hard for humans to read correctly at a glance. "a*b[i]" is better, and "a * b[i]" is the easiest to understand immediately. Control statements are no different. "if (a)if (b)x;" is hard to parse. "if (a) if (b) x;" is better, but "if(a) if(b) x;" leaves no confusion of what belongs where. This recommendation also follows the 'zero one infinity' rule of thumb: https://en.wikipedia.org/wiki/Zero_one_infinity_rule Whether we write "a + b" or "a + b", they are equally readable, and the additional spaces may help with alignment of surrounding expressions. "for (int i : c)" on the other hand makes the keyword unintentionally even more dissociated from its header than "for (int i : c)" already does. The argument that the space helps set it apart from function calls seems moot when practically every editor supports keyword highlighting, function names are typically longer than 2-3 characters, and function calls are not followed by curly brackets (which while optional for singular statements, are still recommended for reasons other than this one). Bug: b/144825072 Change-Id: I3432fadae8e5604123f5c537097323504fecbc8c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39588Tested-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-
Alexis Hetu authored
In order for SwANGLE to expose GL_EXT_float_blend, R32F, RG32F and RGBA32F must all allow blending. The code already supports this, we just need to expose the VK_FORMAT_FEATURE_COLOR_ATTACHMENT_BLEND_BIT bit for these formats. Bug: b/146340613 Tests: dEQP-VK.pipeline.blend.*32_sfloat* Change-Id: I39fc67fa8c26e8296f5e837a8b228a71df9149f1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39631 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:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Also disable the assignment operator implicitly doing a broadcast. Instead the explicit constructor must be used. This prevents bugs like assigning a scalar to one element, while it was intended to broadcast (especially when using [] on an array of vectors). Bug: b/146224130 Bug: b/144825072 Change-Id: I43c977e256e134f6707b5c19252ded08db73b270 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39630 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Bug: b/146224130 Change-Id: Icdbb738ddc286c67a18456529e0137a9d504a8ed Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39629 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>
-
Nicolas Capens authored
This prepares for having other vector sizes (e.g. byte8) support member methods such as broadcast assignment. Note the partial specialization for N=4, to have x,y,z,w members. Bug: b/146224130 Change-Id: I09a341812ddfa14d247e94fd3f9345db951824ed Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39554 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>
-
Nicolas Capens authored
This prepares for simplifying assigning the same scalar to all elements. Bug: b/146224130 Change-Id: I8499c671f98090ae38fd12bab59f30ef360eff69 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39508 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-
Nicolas Capens authored
Since the alignas() specifier is part of the type, it cannot be used in a typedef, and compiler-specific attributes have to be used. We abstracted these into the ALIGN() macro before, but used them in other cases where alignas() can be used. Using it where possible avoids formatting weirdness with clang-format. Bug: b/144825072 Change-Id: If4185df296f0b525dd811103ac5bf1191833b154 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39468 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>
-
Alexis Hetu authored
- Moved the R11G11B10F conversion code from the Blitter to ShaderCore - Used the conversion code in PixelRoutine where appropriate to make the R11G11B10F work as a renderable format - Added the VK_FORMAT_FEATURE_COLOR_ATTACHMENT_BIT and VK_FORMAT_FEATURE_BLIT_DST_BIT to the VK_FORMAT_B10G11R11_UFLOAT_PACK32 format This allows ANGLE to expose the GL_EXT_color_buffer_float (and GL_EXT_color_buffer_half_float, which depends on GL_EXT_color_buffer_float in ANGLE, due a driver issue). Bug: b/146223877 Tests: dEQP-VK.*b10g11r11* Change-Id: I04ad29f7b7d497705186ae290a05868abfc13c42 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39550 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by:
Antonio Maiorano <amaiorano@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
- 13 Dec, 2019 2 commits
-
-
Chris Forbes authored
Bug: b/145348318 Change-Id: I62c8cdf388182a6b30de7bd7d13a49c8dfc1da46 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39528 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
SwiftShader Regression Bot authored
Change-Id: Ief1db6fb999d8e49a3a13ba1e0f56fd13c77dae5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39488 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolas.capens@gmail.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
- 12 Dec, 2019 1 commit
-
-
Sergey Ulanov authored
LLVMReactor was using default memory mapper when creating SectionMemoryManager. The default memory mapper calls mmap() to allocate memory on all platforms except Windows. It doesn't specify MAP_JIT flag that is required to create executable memory on Mac and Fuchsia. As result the allocated memory is not executable, which leads to crashes when trying to run the generated code. Update LLVMRector to use a custom MemoryMapper implementation. Bug: chromium:1032622, b/145348318 Change-Id: Id188af269d80371b709e625c29b92ff546e9ba28 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39408 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-