- 11 Aug, 2017 1 commit
-
-
Nicolas Capens authored
Subzero requires full C++11 support, which isn't available on Android versions prior to Marshmallow. Change-Id: Icf09a51b525a1503f72441c969a9e364306096bd Reviewed-on: https://swiftshader-review.googlesource.com/11488Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 10 Aug, 2017 2 commits
-
-
Alexis Hetu authored
The SwiftShader library, when used to render Layout Tests in Debug, causes too many timeouts. For that reason, we need the Debug version of the library have better overall performance. In Debug, the "default_optimization" defaults to "no_optimization". See: https://cs.chromium.org/chromium/src/build/config/compiler/BUILD.gn?l=1698 In Release, "default_optimization" defaults to "optimize", so in order to at least use the same optimization flags in both case, we replace "default_optimization" by "optimize" in Debug. The Win 7 (dbg) bot ran successfully with this change (the telemetry_tests were failing when building the full Release version of the SwiftShader library with the Debug version of Chromium). Change-Id: I3529160b539319e26f9898d2588536d378c3ee01 Reviewed-on: https://swiftshader-review.googlesource.com/11469Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Nicolas Capens authored
This follows the GLSL ES 1.0 and 3.0 specs more strictly. Change-Id: I323e90ef0a1588109e2cb7988136a9520e501a6d Reviewed-on: https://swiftshader-review.googlesource.com/5065Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 09 Aug, 2017 2 commits
-
-
Nicolas Capens authored
~mask[i % size] is the same as mask[~i % size] because it's just an expansion of i, so there's no need for invMask[] to store the former. Change-Id: I754732a2c4978281d35037941ecb74448ab78a8e Reviewed-on: https://swiftshader-review.googlesource.com/3258Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Branimir Karadžić authored
A timestamp query records the current time when it's issued, not when the data is being retrieved. Change-Id: Idb6e7fe6736a5b54bd23ef1613eaf953da86aa08 Reviewed-on: https://swiftshader-review.googlesource.com/5771Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 08 Aug, 2017 2 commits
-
-
Alexis Hetu authored
This cl simply makes the code compile on Ozone. It does not implement the FrameBufferOzone class, but provides a skeleton for it, which can be implemented by people on the ChromeOS team. Change-Id: Ib77e20b00e8208d992c80f47b5ba55e00254c812 Reviewed-on: https://swiftshader-review.googlesource.com/11348Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
There's an issue with the new debug specific path in the build files. Rolling back these changes in order to do the DEPS roll while the investigation on this issue continues. Change-Id: I9c2174d3ec2ab4e8f817c7cae32f69af1728c6c2 Reviewed-on: https://swiftshader-review.googlesource.com/11328Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
- 05 Aug, 2017 1 commit
-
-
Nicolas Capens authored
This enables swizzling of Int4 and UInt4 types using swizzle operators. Change-Id: I83fe5b472433d84a7b092e18eb78a919fb39b03b Reviewed-on: https://swiftshader-review.googlesource.com/8530Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 04 Aug, 2017 12 commits
-
-
Nicolas Capens authored
Known projects avoid using HAL_PIXEL_FORMAT_RGB_888 because we have no immediate desire to support it (for performance reasons), but other users may run into unexpected issues that previously would leave no trace. Change-Id: Idc606809117f8e0b200b4b38c58ba24696e7193c Reviewed-on: https://swiftshader-review.googlesource.com/11311Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
We currently don't support EGL configs/surfaces with sRGB formats. Change-Id: Ie23a5121bec784fe807c03feae19442fece3f1d7 Reviewed-on: https://swiftshader-review.googlesource.com/11268Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
This should allow to compile on the OZONE platform without hitting the X11 include library problem. The change is imported from: https://cs.chromium.org/chromium/src/third_party/khronos/EGL/eglplatform.h Change-Id: I3b3acc139865797ce6c4173991f15840e91f5935 Reviewed-on: https://swiftshader-review.googlesource.com/11308Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
Updated EGL and KHR headers as is, with one minor patch applied in the ANDROID section of the eglplatform.h files. Other files are unmodified from their Khronos version. Change-Id: I2147df6bf36c7ec4c5cd6d32f8863b512a356112 Reviewed-on: https://swiftshader-review.googlesource.com/11310Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
Switching SwiftShader on ChromeOS from LLVM to Subzero. Change-Id: Ica9ac278272f99b94d365f03fbd7a4860ad3503d Reviewed-on: https://swiftshader-review.googlesource.com/11309Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
Fixed a few types in function signatures so that base and derived classes signatures match. Change-Id: Ide9b9c78ff05be785b512451a36c836f35bd1f3d Reviewed-on: https://swiftshader-review.googlesource.com/5463Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
Change-Id: I4e0e956bd34f5ec2a3c38208cb99ac476ec26623 Reviewed-on: https://swiftshader-review.googlesource.com/11290Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
The functions relating to ANativeWindow have been split off from libandroid into libnativewindow as part of project Treble. Change-Id: I59850e4ba4145e49735698901dc2ba3480c68509 Reviewed-on: https://swiftshader-review.googlesource.com/11229Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Older compiler versions warned about unknown attributes with -Wattributes instead of -Wunknown-attributes. Use both and suppress warnings about unknown warnings options too. Change-Id: I22252426e73d4984471b9b8415479bd7b57ba0c5 Reviewed-on: https://swiftshader-review.googlesource.com/11289Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Forward declare ANativeWindow, and remove <hardware/gralloc.h> where GrallocAndroid.hpp already includes it. Change-Id: Idebd4c40280960ff00cd51b6633c1c84a8cf7de2 Reviewed-on: https://swiftshader-review.googlesource.com/11288Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Older versions of Android using stlport were failing to compile with an "assert has been defined before inclusion of assert.h header" error. Change-Id: If8e13eb08c35a34d082d99f8f57188bc55697f59 Reviewed-on: https://swiftshader-review.googlesource.com/11270Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
This change has been part of the Android NDK since API 18, corresponding with Android 4.3 Jelly Bean MR2. Change-Id: I6df06f46c286ce89ea56913684e43d558917d1a8 Reviewed-on: https://swiftshader-review.googlesource.com/11269Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 03 Aug, 2017 3 commits
-
-
Nico Weber authored
In particular, https://reviews.llvm.org/D36281 Bug: chromium:495204 Change-Id: I48afb5b8dd594ce1b43c6ca9fb20638d40119236 Reviewed-on: https://swiftshader-review.googlesource.com/11248Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nico Weber <thakis@google.com>
-
Alexis Hetu authored
The 'optimize_max' option is not meant to be used by default for Release builds. These use the 'optimize' config. See comment here for explanation: https://cs.chromium.org/chromium/src/build/config/compiler/BUILD.gn?l=1572 Change-Id: I7af8e3e4c7e92c60f38841b75998e2b488a5ffde Reviewed-on: https://swiftshader-review.googlesource.com/11228Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Nicolas Capens authored
This enables using half-float vertex data for OpenGL ES 2.0. Note that GL_HALF_FLOAT_OES does not have the same value as GL_HALF_FLOAT. Change-Id: I0a7a55b7904fe797cdbb86627c8d0a1cc07d2d0a Reviewed-on: https://swiftshader-review.googlesource.com/1500Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 02 Aug, 2017 6 commits
-
-
Alexis Hetu authored
On the mac_optional_gpu_tests_rel bot, a -Wall flag cancels out the -Wno-unused-local-typedef warning suppression and causes the build to fail. Attempting to reorder the configs order to solve the issue. Change-Id: Ic790f5cb7803be4749616d826e1368ef76e2c889 Reviewed-on: https://swiftshader-review.googlesource.com/11212Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
GL_R11F_G11F_B10F was missing from a utility function, causing some dEQP tests to fail. Simply adding it fixes most R11F_G11F_B10F related tests in: functional.fbo.* functional.fragment_out.* functional.pbo.* Change-Id: I0d77f1c9c01aff106e266853e81b4cf45f748bc1 Reviewed-on: https://swiftshader-review.googlesource.com/11211Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
A few more warnings suppression were missing after the last build file change. Added them here to hopefully fix build issues. Change-Id: I80371a3ae59920dccf9c64a36c96d0341e7e39f5 Reviewed-on: https://swiftshader-review.googlesource.com/11210Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
The '-Wall' flag is already specified in Chromium and this '-Wall' flag invalidates any previous '-Wno-*' flags. Removing it solves the compilation issue. Change-Id: Iceefb63ff7e04c55884cf26bbde0d62ffb74b217 Reviewed-on: https://swiftshader-review.googlesource.com/11208Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
SwiftShader's debug version was too slow for Chromium's debug trybots. To solve this issue, SwiftShader now always builds its release version in Chromium. 'git cl format' was ran in Chromium to ensure proper formatting of the gn files. Change-Id: I346045b68b566813ab6ab739aaa6eeb243d51853 Reviewed-on: https://swiftshader-review.googlesource.com/11188Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
We can rely on return value optimization to return Reactor types at the same cost as passing them by reference. Change-Id: I9f092bcefd17131a07bef32032c5f2ab98392328 Reviewed-on: https://swiftshader-review.googlesource.com/10569Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 01 Aug, 2017 8 commits
-
-
Alexis Hetu authored
The GL_RGB10_A2UI was being unpacked as a double sized version of GL_UNSIGNED_SHORT_5_5_5_1, which it is not. The channels are not stored in the same order for these 2 formats. This fixes all "rgb10_a2ui" related test in: dEQP-GLES3.functional.texture.format.sized.* dEQP-GLES3.functional.texture.specification.* Change-Id: I0d398d0537a440906e9ef3aeb01fe4e782f778b6 Reviewed-on: https://swiftshader-review.googlesource.com/11168Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Chromium pulled Subzero into third_party/pnacl-subzero using the recursive DEPS file. We now have the Subzero source code in the SwiftShader repository using Git subtree. Bug swiftshader:47 Change-Id: I2c4c5e8c3413ccd187e4da11fbd014fb76ec572f Reviewed-on: https://swiftshader-review.googlesource.com/9628Reviewed-by:
Corentin Wallez <cwallez@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Bug swiftshader:69 Change-Id: Ib4611e2326741688969137f0610980f899b8ee48 Reviewed-on: https://swiftshader-review.googlesource.com/11148Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
This effectively eliminates all use of MMX instructions, using SSE ones instead. Bug swiftshader:78 Change-Id: I3e519a0c5172688ff711654bf430a9d1dd058d0b Reviewed-on: https://swiftshader-review.googlesource.com/11049Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Chrome and many other products require SSE2 support as a minimum. Note that MMX checks are left in place for now. Dead code paths are removed. Bug swiftshader:78 Change-Id: Iabd8b1dc2092949d5dba29a78e75d014e808f12c Reviewed-on: https://swiftshader-review.googlesource.com/11068Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
This is just a refactoring, where the 'emulated' types still map to the same types as before. They are represented using small integer values reinterpret-casted as sw::Type*, while non-emulated types correspond directly to llvm::Type pointers. Bug swiftshader:78 Change-Id: I88c0b932a45119ce500821068d8fb5faa2ccbb92 Reviewed-on: https://swiftshader-review.googlesource.com/11048Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Using alignment = 0 to signify natural alignment will allow to discern between loads/stores for stack variables, and dereferencing generic pointers. Bug swiftshader:78 Change-Id: I6d9c1728fb9858ca57380bc6bfafc7fb2fa5feae Reviewed-on: https://swiftshader-review.googlesource.com/10968Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
The Optimizer used to assume that loads and stores using the same address are of the same tpe. This is false for emulated sub-vector load/store, and thus we tricked it into thinking the addresses are different by performing some redundant arithmetic. This is no longer necessary now that the Optimizer checks the types being loaded/stored. Bug swiftshader:48 Change-Id: Ic212d67fefd9e8dd902ff576b69458208c3c379d Reviewed-on: https://swiftshader-review.googlesource.com/11088Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
- 28 Jul, 2017 1 commit
-
-
Alexis Hetu authored
The bitwise NOT operation is supported for integer types in GLSL. All the plumbing was already there, only the actual call was missing. In dEQP, all shaders.operator.unary_operator.bitwise_not tests pass. Change-Id: I938104b860441f6b2df6c807d578f651e5cd2516 Reviewed-on: https://swiftshader-review.googlesource.com/11108Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 27 Jul, 2017 1 commit
-
-
Alexis Hetu authored
This cl implements support for gl_VertexID. Passes the functional.shaders.builtin_variable.vertex_id test. Change-Id: I5550e3ecba30e29f1e38ace608d730833a1e9598 Reviewed-on: https://swiftshader-review.googlesource.com/10958Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 26 Jul, 2017 1 commit
-
-
Nicolas Capens authored
The SignMask intrinsic returns the most significant bit of each byte in a 128-bit vector. For 'emulated' 8-byte vectors we expect the upper half to be ignored. Change-Id: I994e35fb9c21710c35a89add42297c3c9bad07a4 Reviewed-on: https://swiftshader-review.googlesource.com/10988Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-