- 18 Dec, 2019 4 commits
-
-
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:
-
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:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
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:
-
- 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 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 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 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 authored
Bug: b/144825072 Change-Id: I869aef91d6318bf6955581e5dad762800bd46296 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39655Tested-by:
-
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 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:
-
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:
-
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:
-
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:
-
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 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:
-
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 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:
-
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:
-
- 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:
-
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:
-
- 12 Dec, 2019 5 commits
-
-
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:
-
Alexis Hetu authored
There was a missing prepareForSampling call after doing an image copy between 2 compressed textures. This cl fixes these angle end to end tests: CopyCompressedTextureTest.Basic/ES3_Vulkan_SwiftShader CopyCompressedTextureTest.Basic/ES2_Vulkan_SwiftShader Bug: b/146052572 b/146157683 Change-Id: I8703fa71557509e9ecb0d52a4ad2da4c72fa6812 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39448 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by:
-
Nicolas Capens authored
The expression "IfThenElse(e < 24, mantissa >> e, Int(0))" gets rewritten into "IfThenElse(e<24, mantissa> > e, Int(0))" by some versions of clang-format, thus breaking the build. It's likely a clang-format bug that should be fixed, but the workaround is simple and still quite elegant: "IfThenElse(e < 24, (mantissa >> e), Int(0))" Bug: b/144825072 Change-Id: I9cc19c6cae31bb0452b4e5402f6a742d15c45e55 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39428 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Reviewed-by:
-
Alexis Hetu authored
Added a block compression (BCn) decoder and logic for all formats for n in the [1, 5] range. The decompression follows the same logic as the ETC2 decompression in vk::Image. Tests: dEQP-VK.*bc* Bug: b/146052572 Change-Id: I64fac0a7af52e1be209c1cfd5373744918c7df14 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39369 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Ben Clayton authored
This block declares variables, so should be scoped. Bug: b/145351270 Change-Id: I90b750a81f4ff93f1c618a2ae969fa321961673c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39311Tested-by:
-
- 11 Dec, 2019 10 commits
-
-
Ben Clayton authored
Server implements a Debug Adapter Protocol server that listens on a specific port, that operates on the vk::dbg::Context passed to the constructor. Bug: b/145351270 Change-Id: If09d92c6d298cfbd71a7b2366b6b8407a728ec43 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38899Reviewed-by:
-
Ben Clayton authored
Holds the global debugger state. Bug: b/145351270 Change-Id: I4e483c145af068755716f23c09adcc68d138ceaa Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38898Reviewed-by:
-
Ben Clayton authored
This CL contains minor tweaks to the style of our code to make clang-format produce prettier code. Bug: b/144825072 Change-Id: I821ecd3b9f408a4034992c2cc7ece2ef8552fe08 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39314Tested-by:
-
Antonio Maiorano authored
Bug: swiftshader:142 Change-Id: I188fb7895f5f52c4d5eb85ca5a4fc091ee95b604 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39368Reviewed-by:
-
Ben Clayton authored
`err` was getting nullified when the program terminated abnormally, preventing the regex matching logic from being called. Should fix binning of abortion caused by SwiftShader debug macros. Change-Id: I811441296841c77cb87839039cde754d13f2a108 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39309 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Paul Thomson <paulthomson@google.com> Reviewed-by:
-
Nicolas Capens authored
This is a style change. Visual Studio defaults to indenting namespace contents, and this was adopted for a long time, but with the new Vulkan implementation this was abandoned. However the legacy code borrowed from the OpenGL ES implementation still used indentation so it was inconsistent. The justification for not indenting namespace contents is that namespaces are merely a way to avoid name clashes with other projects we don't control directly (and in rare cases internal subprojects when we want to reuse the same names). Hence the vast majority of files have a single namespace, and unlike indentation used for ease of discerning control flow blocks, class contents, or function contents, which can become highly nested, there is no such readability advantage to indenting namespace contents. This is also the Google style recommendation (though no justification or discussion is provided): https://google.github.io/styleguide/cppguide.html#Namespace_Formatting One reasonable counter-argument is consistency with other blocks of curly brackets, but considering that most namespaces span almost the entire file, it's a substantial waste of line length. Because there is no indentation, there's also no need to have the open and closing brackets line up as a visual aid, like we prefer for other uses of curly brackets. So we place the open bracket on the same line as the namespace keyword. A comment is added to the closing bracket to discern it from other closing brackets. It also makes it easier to find the end of anonymous namespaces which typically go at the top of the source file. This change is make separately from applying clang-format because diff tools mark all these unindented lines as changes and this makes it hard to review the smaller style changes made by clang-format. The OpenGL ES and Direct3D code is left untouched because it is in maintenance mode and in case of regressions we want easy 'blame' tool usage. Bug: b/144825072 Change-Id: Ie2925ebd697e1ffa7c4cbdc9a946531f11f4d934 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39348 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Reviewed-by:
-
Ben Clayton authored
These will be used to create a synthetic file for all the commands in the command buffer, which can be stepped over using the new debugger. Bug: b/145351270 Change-Id: If5728463c1cf333d635c46e2d3b6314ce7d27326 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38910Tested-by:
-
Ben Clayton authored
Also add SPIRV-Tools, as this was missing and caused red squiggles. Bug: b/145351270 Change-Id: I22e664bb662f75284cf93cf6e446e775521c7b26 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38908Tested-by:
-
SwiftShader Regression Bot authored
Change-Id: I4c2ca30b1cc9edda07c83056d9798fe8c202152b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39313Reviewed-by:
-
Nicolas Capens authored
The Subzero JIT now has sufficient functionality to support running the Vulkan unit tests. Bug: b/130461828 Change-Id: I8c398b976899151ac4a82549df21e06127a115f1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39328Tested-by:
-
- 10 Dec, 2019 4 commits
-
-
Ben Clayton authored
Both code paths will be tested for Windows, macOS, Linux for both LLVM and Subzero backends. Bug: b/145836207 Change-Id: I707b1551fe1a65549c47a6d7756ab94aeb8d80e2 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39312 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Chris Forbes authored
VkPhysicalDeviceFeatures::imageCubeArray implies both the ImageCubeArray and SampledCubeArray SPIRV capabilities are supported. We had only accepted SampledCubeArray so far. Bug: b/144988970 Change-Id: I93fb93a95047f73c677de38757660ea1321a4654 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39289Tested-by:
-
Ben Clayton authored
EventListener is an interface that is used to listen for thread events. Thread holds the state for a single thread of execution. Frame holds a number of variable scopes for one of a thread's stack frame. Scope is a container for frame variables. Bug: b/145351270 Change-Id: Ic61e17f32cfd6929dbd7b0fce1ffb716301fc73e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38897Reviewed-by:
-
Ben Clayton authored
It holds a file path and line number. Bug: b/145351270 Change-Id: I0f54d00bda151be529d912562061052abff2fe81 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38896Reviewed-by:
-
