- 02 Oct, 2018 1 commit
-
-
Nicolas Capens authored
SwiftShader shouldn't print anything to stdout in Release builds. Also refactor UNIMPLEMENTED macro to be able to take a formatted string with arguments. Bug b/73656151 Change-Id: Ibadbfba3371324ba1bd608bd51bdac5e5923a20e Reviewed-on: https://swiftshader-review.googlesource.com/21108Reviewed-by:
Merck Hung <merckhung@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 01 Oct, 2018 2 commits
-
-
Nicolas Capens authored
Bug b/115344057 Change-Id: Iac34e89f56a1f67301beae3fcf492596522d08a2 Reviewed-on: https://swiftshader-review.googlesource.com/21168Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
Nicolas Capens authored
Locking stencil buffers with a NULL format resulted in nullptr being returned without acquiring the resource lock, while unlocking did release the lock. Instead of not releasing it on unlock, we must acquire it on lock because otherwise the object could get destroyed before draw operations end and attempt to unlock. Bug b/116876483 Change-Id: Ie883115a1d6df3b40eb7c1feb5dcfde7316ec970 Reviewed-on: https://swiftshader-review.googlesource.com/21148Reviewed-by:
Krzysztof Kosiński <krzysio@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Krzysztof Kosiński <krzysio@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 28 Sep, 2018 3 commits
-
-
Nicolas Capens authored
Bug b/115784742 Change-Id: Id3225adcaf77e1d26560151c506e885318bd0ff6 Reviewed-on: https://swiftshader-review.googlesource.com/21088Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
Merck Hung authored
Replace all-makefiles-under makefile-macro with explicitly including Android.mk(s) from subdirectories that are related to OpenGL and are only needed by Android builds. The change are mainly made to the top-level Android.mk and the src/Android.mk. The other three sub-Android.mk(s) are deprecated and deleted. Soong compile-time is expected to shorten with no use of all-makefiles-under macros. Since the project co-existed with both Android.bp and Android.mk. The test was performed in absence of Android.bp on oc-mr1-dev branch against aosp_arm-eng and aosp_arm64-eng targets. NOTE: Android Pie uses Android.bp only. Bug: b/29023322 Test: oc-mr1-dev branch, aosp_arm-eng (LLVM3) & aosp_arm64-eng (LLVM7) targets Change-Id: I05ac46213f849747621c7084bca2549a22513881 Reviewed-on: https://swiftshader-review.googlesource.com/21068Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Merck Hung <merckhung@google.com>
-
Nicolas Capens authored
The spec for glDeleteVertexArrays states that "Unused names in arrays are silently ignored, as is the value zero." b/116699321 Change-Id: I7cca0336dd9841b360f8fa20f6c0ac9677ec3ce1 Reviewed-on: https://swiftshader-review.googlesource.com/21048Reviewed-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 27 Sep, 2018 9 commits
-
-
Logan Chien authored
This commit fixes `Trunc(Float)` and `Trunc(Float4)` generic LLVM code generation. If `Trunc(x)` is implemented with `Float(Int(x))`, it will result in quality warning in dEQP. Bug: b/115344057 Test: dEQP-GLES3.functional.shaders.builtin_functions.precision.trunc Test: dEQP-GLES3.functional.shaders.builtin_functions.precision.modf Change-Id: I62d2dd1907e345fb00307b6c0d4d74613237f94b Reviewed-on: https://swiftshader-review.googlesource.com/21029Tested-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
Logan Chien authored
This commit fixes `Frac(Float)` and `Frac(Float4)` generic LLVM code generation. See also: https://bugs.chromium.org/p/swiftshader/issues/detail?id=74 Bug: b/115344057 Test: functional.shaders.builtin_functions.precision.fract.highp_vertex Test: functional.shaders.builtin_functions.precision.fract.highp_fragment Change-Id: I027b7ab44ba3060dc100a220ba19e5275f41d4c9 Reviewed-on: https://swiftshader-review.googlesource.com/20933Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Logan Chien authored
This commit fixes `RoundInt(Float)` and `RoundInt(Float4)` generic LLVM code generation. Bug: b/115344057 Test: functional.shaders.builtin_functions.precision.sinh.highp_vertex Test: functional.shaders.builtin_functions.precision.sinh.highp_fragment Test: functional.shaders.builtin_functions.precision.cosh.highp_vertex Test: functional.shaders.builtin_functions.precision.cosh.highp_fragment Change-Id: I35de5cf6bec6607725523a10ebf098333957f39a Reviewed-on: https://swiftshader-review.googlesource.com/20931Tested-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
Logan Chien authored
This commit fixes `Round(Float)` and `Round(Float4)` generic LLVM code generation. Bug: b/115344057 Test: dEQP-GLES3.functional.shaders.builtin_functions.precision.sin Test: dEQP-GLES3.functional.shaders.builtin_functions.precision.cos Change-Id: I9758ac4ca69bb0f5a5b192b4be3310827625fac0 Reviewed-on: https://swiftshader-review.googlesource.com/20932Tested-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
Logan Chien authored
Bug: b/115344057 Change-Id: I57d9cc936349c9f788a9bf791f55958caa6cbb73 Reviewed-on: https://swiftshader-review.googlesource.com/20930Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Logan Chien authored
This commit fixes Floor() generic LLVM code generation. Bug: b/115344057 Test: dEQP-GLES3.functional.shaders.builtin_functions.precision.floor Change-Id: I70ec0babfe9778b7963296734f1901a73b1f696f Reviewed-on: https://swiftshader-review.googlesource.com/20929Tested-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
Logan Chien authored
This commit fixes saturated add/sub instructions in generic LLVM code generation path. Bug: b/115344057 Test: dEQP-GLES3.functional.texture.wrap.rgba8 Change-Id: Ie3e3b708565b3ad255804090e8a3ee5521f42982 Reviewed-on: https://swiftshader-review.googlesource.com/20928Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Logan Chien authored
Use mixed indentations so that the code are aligned regardless the tab width. Bug: b/115344057 Change-Id: I9476c1b8ba5a518787dc76f50e1805b1b9d06e82 Reviewed-on: https://swiftshader-review.googlesource.com/21028Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Merck Hung authored
Change-Id: I4e8a860446bde7f76ea289888bf592d78c59b9c4 Reviewed-on: https://swiftshader-review.googlesource.com/21008Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Merck Hung <merckhung@google.com>
-
- 26 Sep, 2018 4 commits
-
-
Chris Forbes authored
Change-Id: I34b47a5327055f804951793d9a8a58f771d9b792 Bug: b/116336664 Reviewed-on: https://swiftshader-review.googlesource.com/20988Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Nicolas Capens authored
Bug chromium:101600 Bug swiftshader:118 Change-Id: Id167a84c4d8781989d4d903384c4e6fe6f45fb85 Reviewed-on: https://swiftshader-review.googlesource.com/20868Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
This patch contains an empty Vulkan shell with only GetProcAddr implemented Initial version of the VulkanAPI, empty and unimplemented. Successfully builds vk_swiftshader.dll for all platforms/configuration in Visual Studio. To test using dEQP: - Edit environment variables - Define VK_ICD_FILENAMES to <SwiftShader's source directory>\src\Vulkan\vk_swiftshader_icd.json - If the location of vk_swiftshader.dll you're using is different than the one specified in src\Vulkan\vk_swiftshader_icd.json, modify it to point to the vk_swiftshader.dll file you want to use Bug b/116336664 Change-Id: I560b53c3e4340b9c5ccd244a6693ff2f9f994a6f Reviewed-on: https://swiftshader-review.googlesource.com/20788Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Logan Chien authored
This commit switches several internal (implied) Makefile variables to lower cases. This follows the variable naming convention recommended by GNU Makefile Manual. Change-Id: I79819ef0cfd3828107dab665118f1511ef8a8337 Reviewed-on: https://swiftshader-review.googlesource.com/20848Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Logan Chien <loganchien@google.com>
-
- 25 Sep, 2018 2 commits
-
-
Chris Forbes authored
Tweak ContextPtr to handle move construction/assignment correctly. This was overlooked when I made the last minute change to use explicit lock/unlock. Tweak LockGuard to allow construction from ptr to MutexLock, and behave correctly if mutex == nullptr. This is required because the locking machinery for EGL is /outside/ the error generation for bad EGLDisplay. Add 'Big EGL Lock' around most of the EGL API Bug: b/112184433 Change-Id: Ic4cd5fa9e6a16ae81cbb2b46e1cb881a850749c5 Reviewed-on: https://swiftshader-review.googlesource.com/20628Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Chris Forbes authored
Fixes: dEQP-GLES3.functional.shaders.constant_expressions.other.complex* Bug: b/116598062 Change-Id: Ife2dd2edad17d6ebbd02653f484637672ce6af0a Reviewed-on: https://swiftshader-review.googlesource.com/20828Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
- 21 Sep, 2018 5 commits
-
-
Chris Forbes authored
Bug: b/116263076 Change-Id: Id8657fdefb988827f0d78556884c4f55eb096ca3 Reviewed-on: https://swiftshader-review.googlesource.com/20808Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Chris Forbes authored
As mentioned in the patch itself, this is fairly benign, but still junk that a later pass has to be able to eliminate. Is easier to just not produce it in the first place. Bug: b/116263076 Change-Id: I199672bb9b53eb4ee9d4a8e3a23fc2ce42f0dec6 Reviewed-on: https://swiftshader-review.googlesource.com/20770Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Chris Forbes authored
Matrix is laid out in column-major order, so the stride between elements on the main diagonal is #rows + 1, not #columns + 1. Bug: b/116263076 Change-Id: I00994b663bc4229e24f8ae1b7fa518caa0e7ea71 Reviewed-on: https://swiftshader-review.googlesource.com/20768Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Chris Forbes authored
Cleans up some noise that appears in dumps of pretty much every GLSL shader. Change-Id: I602a6356110803e30fcd8ba9818e56efa70bf5d6 Reviewed-on: https://swiftshader-review.googlesource.com/20769Tested-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Chris Forbes authored
Khronos test CL: https://gerrit.khronos.org/#/c/2944/ Bug: b/116259485 Change-Id: Ie10b3eb16da39f332966a5b957acb0232140a942 Reviewed-on: https://swiftshader-review.googlesource.com/20748Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
- 20 Sep, 2018 1 commit
-
-
Logan Chien authored
This commit replaces use_subzero with SWIFTSHADER_USE_SUBZERO and unsupported_build with SWIFTSHADER_UNSUPPORTED_BUILD in Android.mk so that it can be consistent with other variables. Change-Id: Ia22cc88bc5804d7d2bea942bbceb2fc30c88aa18 Reviewed-on: https://swiftshader-review.googlesource.com/20588Tested-by:
Logan Chien <loganchien@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 14 Sep, 2018 1 commit
-
-
Alexis Hetu authored
Made sure both LLVM and Subzero has the /src directory in their include directory list. Removed unused ../ in front of "Common" includes as a cleanup. Change-Id: Ib4a6fb772fa03a9a3cad82594db1b1650007e47f Reviewed-on: https://swiftshader-review.googlesource.com/20728Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
- 13 Sep, 2018 2 commits
-
-
Nicolas Capens authored
Failures in dEQP using SwiftShader with LLVM 7.0 were caused by unsafe floating point math optimizations, which are more aggressive than with LLVM 3.0. Bug b/115344057 Change-Id: Idf564d0193efba37cd1192a188b18a5f8b4ed4f7 Reviewed-on: https://swiftshader-review.googlesource.com/20708Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Shannon Woods <shannonwoods@google.com>
-
Nicolas Capens authored
LLVM fails to resolve the __chkstck function for probing the stack when it's larger than 4 kB. We don't need it for SwiftShader because newly created threads have 1 MB of committed stack memory. Bug b/115344057 Bug swiftshader:25 Change-Id: Ic6097376052b07ed6af950dfdac8499b5fc83053 Reviewed-on: https://swiftshader-review.googlesource.com/20688Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 12 Sep, 2018 2 commits
-
-
Shannon Woods authored
Change-Id: Ia4b0f5b7911ddc1f425430e2f6b2e2a7c80fb684 Reviewed-on: https://swiftshader-review.googlesource.com/20668Tested-by:
Shannon Woods <shannonwoods@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
These were created by creating a Visual Studio solution for LLVM itself and building it so it generates include\llvm\Config files. Bug b/115344057 Change-Id: I22c93d63395fa27b8d8515d65c6bf4e09b3c16bb Reviewed-on: https://swiftshader-review.googlesource.com/20608Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
- 11 Sep, 2018 8 commits
-
-
Stephen Lanham authored
Bug: b/114402930 Change-Id: Id7eedc6b01f40d24188d514755d5c89689b24abb Reviewed-on: https://swiftshader-review.googlesource.com/20568Tested-by:
Stephen Lanham <slan@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Logan Chien authored
Change-Id: Ic1ae2c2872cc9f270557a6d4b701f870f82bd3cf Reviewed-on: https://swiftshader-review.googlesource.com/20479Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Logan Chien <loganchien@google.com>
-
Logan Chien authored
Change-Id: Id4209bfe44d9040e36fe4ea654197211aa1c7ead Reviewed-on: https://swiftshader-review.googlesource.com/20478Tested-by:
Logan Chien <loganchien@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Logan Chien authored
Change-Id: Id5ae40d4b072c57cac1a8e6e7767d3ca03deebe1 Reviewed-on: https://swiftshader-review.googlesource.com/20477Tested-by:
Logan Chien <loganchien@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Logan Chien authored
This commit fills in the unimplemented non-x86 (generic) code generator. Change-Id: I189b0ea523ecd9b18c29ad6ed6fa3f798382295b Reviewed-on: https://swiftshader-review.googlesource.com/20476Tested-by:
Logan Chien <loganchien@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Logan Chien authored
Change-Id: I89601ef9e42e56cfaeb1cd5ece4daa3f32b39950 Reviewed-on: https://swiftshader-review.googlesource.com/20475Tested-by:
Logan Chien <loganchien@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Logan Chien authored
Change-Id: Icbacb2f63402b1d1c42fc4287b50ed0a3010cd4d Reviewed-on: https://swiftshader-review.googlesource.com/20474Tested-by:
Logan Chien <loganchien@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Logan Chien authored
Change-Id: Ic67a43f5090c5004324ab300ce799c1206f3c4e3 Reviewed-on: https://swiftshader-review.googlesource.com/20473Tested-by:
Logan Chien <loganchien@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-