- 23 Oct, 2015 1 commit
-
-
Alexis Hetu authored
Added new read/write functionality for integer types. Also added cases for integer types in some utility functions. Change-Id: I5efc7c4957d3a1591b47a5df888a9534776033be Reviewed-on: https://swiftshader-review.googlesource.com/4114Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 22 Oct, 2015 2 commits
-
-
Nicolas Capens authored
Bug 25161109 This reverts commit 810f99be. It was only a workaround, which we no longer need after removing the libandroid dependency which caused circular symbol resolves. Change-Id: I0b486b5acfd328c0f64379e2c2dc2db00626738f Reviewed-on: https://swiftshader-review.googlesource.com/4130Tested-by:
Nicolas Capens <capn@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
The blitter now supports integer types. This means: - It supports conversions to/from integer types from/to other already supported types. - It supports integer to integer conversions without going to an intermediate float format. Also added a Blitter::GetScale() function to avoid having 2 instances of the same switch statement in the code and added the read/write utility functions. The final Blitter code is not longer peppered with switch statements and is, hopefully, easier to read that way. Change-Id: I80de519aaaa768f8cedd98f97dc4414dda75bf54 Reviewed-on: https://swiftshader-review.googlesource.com/4113Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 21 Oct, 2015 9 commits
-
-
Greg Hartman authored
This was causing dlsym to return the pointer to the wrapper's function when SwiftShader didn't support an entry point. Bug 25152138 Change-Id: I92d1c9258c823acf0dc60dca9dc3d7dd552512b3 Reviewed-on: https://swiftshader-review.googlesource.com/4120Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Nicolas Capens <capn@google.com> Tested-by:
Greg Hartman <ghartman@google.com>
-
Nicolas Capens authored
Change-Id: I604b2f755ece6a0a411562e9229a39040b3b0153 Reviewed-on: https://swiftshader-review.googlesource.com/4112Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
Added initialization for swizzle related members of Sampler Change-Id: I81f7aad5274696390e73c32ff1b424687e9f5943 Reviewed-on: https://swiftshader-review.googlesource.com/4111Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
Added new enums for: - 8/16/32 bit integers - R, RG, RGB, RGBA - Normalized and unnormalized Also fixed FORMAT_LAST. Change-Id: Ie3eb5286820c44490630e302907350358a1d85dc Reviewed-on: https://swiftshader-review.googlesource.com/4110Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
Initial TextureSize parsing and implementation Change-Id: I8b9b1808366b1013a5001e2dfa15a26d8471ab6a Reviewed-on: https://swiftshader-review.googlesource.com/3753Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Nicolas Capens authored
Bug 24111119 This is a trivial implementation which does not actually optimize compositing using the damage rects. That can be implemented later. Change-Id: Ibf511ef166a33b1d2f1826bfc5618765062fe2a2 Reviewed-on: https://swiftshader-review.googlesource.com/4014Tested-by:
Nicolas Capens <capn@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Nicolas Capens authored
Bug 21662268 Change-Id: Iefdc18303c42967ccc25fd6580851e05520dc2b9 Reviewed-on: https://swiftshader-review.googlesource.com/4071Tested-by:
Nicolas Capens <capn@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
The new parameters are: swizzleR, swizzleG, swizzleB, swizzleA. Also, parameters sent from the context that can be either originating from the Texture object or the Sampler object are now sent from the correct origin. An implementation for the swizzleR, swizzleG, swizzleB, swizzleA parameters was done in SamplerCore. All related dEQP tests pass. Change-Id: I45405a0d241d0e70a91a3c56357c7bc04c7dc75a Reviewed-on: https://swiftshader-review.googlesource.com/4017Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
Mipmap levels used in texture sampling must be limited by the texture's base level (default: 0) and max level (default: 1000). This has an effect on texture completeness, since only the required levels are required to be present in the texture to be considered complete. Change-Id: I31dd87d5a4306622e469c2546107b59f6e51c7dd Reviewed-on: https://swiftshader-review.googlesource.com/4039Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 15 Oct, 2015 2 commits
-
-
Alexis Hetu authored
The unpack row length was supposed to replace the width, not the pitch, of the input image, which explains why related tests were failing. Change-Id: I7bd46e057bae4113b31d072fd6748d9dd6043feb Reviewed-on: https://swiftshader-review.googlesource.com/4080Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
Added 2 new constructors for the Int4 class in Nucleus, which are for conversion to Int4 from Short4 or UShort4. Some of the required code was extracted from Float4 constructors, which were cleaned up to remove the duplicate code. All 4 constructors (2 new Int4 and 2 modified Float4) behaved properly in dEQP tests. Change-Id: Ib400ea27a867d31901f4a9ca08bf6093f186b277 Reviewed-on: https://swiftshader-review.googlesource.com/4060Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 08 Oct, 2015 2 commits
-
-
Greg Hartman authored
Change-Id: I12c262fbdc2b2b489c02871fb9477bf25a0cfb6b Reviewed-on: https://swiftshader-review.googlesource.com/4050Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Greg Hartman <ghartman@google.com>
-
Greg Hartman authored
Change-Id: I14673f17ef0f299a7ead5bc55bd6729f37759b19 Reviewed-on: https://swiftshader-review.googlesource.com/3870Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Greg Hartman <ghartman@google.com>
-
- 01 Oct, 2015 2 commits
-
-
Alexis Hetu authored
- A new routine has been added to convert from any component of 8bit sRGB data to 8bit linear RGB, using a precomputed array. - Two new classes have been added to easily convert to and from the RGB9E5 format and the R11G11B10F format. Change-Id: I85ca58bed30bcd5a9130bca5040d351badabb19e Reviewed-on: https://swiftshader-review.googlesource.com/3990Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Nicolas Capens authored
Change-Id: I201ffae8193cfcac48a7ad3446f4b79da6c3d217 Reviewed-on: https://swiftshader-review.googlesource.com/4032Tested-by:
Greg Hartman <ghartman@google.com> Tested-by:
Nicolas Capens <capn@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 30 Sep, 2015 2 commits
-
-
Alexis Hetu authored
A new ETC2 decoder was added to SwiftShader, based on the OpenGL ETC2 specification. The decoder is fully standalone and does not rely on any outside code (there are no files included in the header and the source file only include the header file, so it can easily be ported to any other project). Things to note: - In Surface.cpp, signed ETC2 images are decoded to full 32FP images, because of the lack of support for signed 8 bit R and RG internal formats. This should be fixed as soon as these formats are made available. - sRGB conversion is not performed within the decoder, so it has been added as a loop inside Surface::decodeETC2 after the ETC2 decoding is performed. This is to make sure that there is no loss of precision, should we choose to do the conversion to a higher bit precision format. The loop is fairly straightforward and does the conversion in place, so the impact on performance compared to doing the sRGB conversion in the decoder should be minimal. Change-Id: I3a1af623353344bf35818ba9c9f4cf349b587e2f Reviewed-on: https://swiftshader-review.googlesource.com/3960Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
Implementation for determinant has been done directly in ShaderCore in order to avoid having to allocate temporaries manually in OutputASM. For now, the implementation for the inverse matrix is very simple, i.e., it doesn't attempt to re-use results from the cofactor matrix computation to compute the determinant or do any other kind of optimization, but it works. Change-Id: I0fc70133809ae2752dc567bf58b60d7af7a88009 Reviewed-on: https://swiftshader-review.googlesource.com/4000Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
- 29 Sep, 2015 1 commit
-
-
Alexis Hetu authored
A few cases of matrix multiplication weren't working properly due to using the wrong matrix dimension. For example, if we have the following matrix operation (using matColRow): mat32 = mat22 * mat32 This results in 3 multiplication of 2 element vectors for each row of the result and the current code (before this cl) would only perform 2 multiplications and produce the wrong result. This cl should fix all uses of the * operator and of the matrixCompMult function when using non square matrices. Change-Id: Id0dbfd9d65c20102220049c34435c37e3db7f9da Reviewed-on: https://swiftshader-review.googlesource.com/3966Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 28 Sep, 2015 1 commit
-
-
Alexis Hetu authored
Fixed 2 occurrences of setCompareMode being used instead of setImmutableFormat and added a missing break statement. Change-Id: I1013d0983220ba155dcd4f6255c6e089f03e7fc8 Reviewed-on: https://swiftshader-review.googlesource.com/4016Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 25 Sep, 2015 1 commit
-
-
Alexis Hetu authored
Some issues, like failures using the do-while command, were caused by the assembler being unable to assign registers consistently for a single temporary variable. For example, in the do-while case, the "iterate" Temporary object was not being assigned to the same register between it's initial declaration and it's usage inside the loop condition later on, causing the do-while to behave weirdly. Other instances where multiple Temporary objects ended up being used simultaneously of where a single object was references in multiple parts of the code could have failed because of this. The fix is simply to assign Temporary objects a unique ID. Change-Id: Ie48c596b4c6570853702cab71497b3e00a40c2a0 Reviewed-on: https://swiftshader-review.googlesource.com/4015Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 24 Sep, 2015 3 commits
-
-
Nicolas Capens authored
Bug 20891368 Change-Id: Ie293159d2092985c496202eb25733e69f9a3ff6f Reviewed-on: https://swiftshader-review.googlesource.com/3252Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Greg Hartman <ghartman@google.com> Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Nicolas Capens <capn@google.com>
-
Nicolas Capens authored
Bug 20891368 Change-Id: Iea526eebe65616079578563126a6958d87647eb1 Reviewed-on: https://swiftshader-review.googlesource.com/3255Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Nicolas Capens <capn@google.com>
-
Nicolas Capens authored
Change-Id: I4f610396af2d31792d9b9e452abd8ac597eb1aee Reviewed-on: https://swiftshader-review.googlesource.com/4011Reviewed-by:
Greg Hartman <ghartman@google.com> Tested-by:
Greg Hartman <ghartman@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 23 Sep, 2015 3 commits
-
-
Alexis Hetu authored
- Removed redundant parameters from fogBlend() - Moved computation registers and color output registers from PixelRoutine to PixelProgram. - Made many PixelRoutine member functions private rather than protected when possible. Change-Id: I748333626d993c0d46b369991d74bc3a22c972c2 Reviewed-on: https://swiftshader-review.googlesource.com/3850Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
Bug 20724899 Change-Id: I03f858ee3d159cfa2541afa1d8d063ca99701a4c Reviewed-on: https://swiftshader-review.googlesource.com/3964Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
uint uniforms were failing because of a missing condition. This only affected uint scalars, uint vectors were unaffected. Change-Id: Ia2338a683f1e6f5f9b6944aae72e2dbec70a8ef2 Reviewed-on: https://swiftshader-review.googlesource.com/3963Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 22 Sep, 2015 4 commits
-
-
Nicolas Capens authored
Bug 20891368 Change-Id: I29054ccc0a91fdc41d26d26dd4f55dfd4dfca7e4 Reviewed-on: https://swiftshader-review.googlesource.com/3952Tested-by:
Nicolas Capens <capn@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Nicolas Capens authored
Change-Id: Ib9dadf3d8fee0f63deb10e6754856c0530c928ab Reviewed-on: https://swiftshader-review.googlesource.com/3995Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Nicolas Capens <capn@google.com>
-
Nicolas Capens authored
Bug 22375329 Change-Id: Ifeb8017d50224b784dedd6197779c6ebfa5cf484 Reviewed-on: https://swiftshader-review.googlesource.com/3994Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Nicolas Capens <capn@google.com>
-
Nicolas Capens authored
Bug 20891368 Change-Id: Icc0957db55cf82e42c0ccb2c8881332f71f52f5f Reviewed-on: https://swiftshader-review.googlesource.com/3993Tested-by:
Nicolas Capens <capn@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
- 21 Sep, 2015 5 commits
-
-
Greg Hartman authored
Change-Id: If2fb1ba3ab1428086ec35bc5824e4307156c67a0 Reviewed-on: https://swiftshader-review.googlesource.com/3980Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Greg Hartman <ghartman@google.com>
-
Alexis Hetu authored
glBindSampler's sampler check was too strict, as the sampler needed to exist, but not necessarily to have been allocated yet, so the condition was fixed. Change-Id: Ia8301028dbf35efa16bfc6cb74a69e105d8a505d Reviewed-on: https://swiftshader-review.googlesource.com/3965Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
- Transform feedback buffers are now properly cleared when the object is deleted - Transform feedback buffers are now properly detached when a buffer is deleted - Ending a transform feedback now properly sets its PAUSED state to false - Added missing enums to Context::getQueryParameterInfo. GL_READ_FRAMEBUFFER_BINDING_ANGLE was also missing, so it was also added in this cl. Change-Id: I286caec119d5ae3adde95c4cce7f0298b8f836c3 Reviewed-on: https://swiftshader-review.googlesource.com/3962Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com>
-
Alexis Hetu authored
- Fixed a few typos in the format map - Added missing OpenGL ES 3.0 types to CheckTextureFormatType Change-Id: I4a0d8ad67fd85f8c32ef528ce1b3c029fe5f84aa Reviewed-on: https://swiftshader-review.googlesource.com/3991Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
Change-Id: I9e70cd5e230a86ef3e4bed1884ede14a128c2d67 Reviewed-on: https://swiftshader-review.googlesource.com/3992Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
- 11 Sep, 2015 1 commit
-
-
Greg Hartman authored
Change-Id: I859db3be12c24fc4544dc3780e5fdf5c598412d8 Reviewed-on: https://swiftshader-review.googlesource.com/3970Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Greg Hartman <ghartman@google.com>
-
- 03 Sep, 2015 1 commit
-
-
Nicolas Capens authored
Bug 23041739 Change-Id: I1bec1fa05e8cf81864f8c0c2f49cf9c8093de5e0 Reviewed-on: https://swiftshader-review.googlesource.com/3934Reviewed-by:
Greg Hartman <ghartman@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <capn@google.com> Tested-by:
Nicolas Capens <capn@google.com>
-