Change-Id: Ic229ced16f667882521536db5bd483c76b24d600
Reviewed-on: https://chromium-review.googlesource.com/242905Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

Change-Id: Iab44f6d634bdcd01348aa49de28b96c3b6086df7
Reviewed-on: https://chromium-review.googlesource.com/241945Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Austin Kinross <aukinros@microsoft.com>

Change-Id: Ibcb72f7b1aabcd98c4bebd91c3a57f5ba57eac3a
Reviewed-on: https://chromium-review.googlesource.com/242850Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

D3D11 Feature Level 9_3 can't disable mipmaps on a mipmapped
texture, and sample from level zero of it. A previous commit
added a workaround for this in ANGLE to Texture2Ds. This
commit fixes some minor issues in that commit, and extends
the workaround to apply to TextureCubes too.

Change-Id: Ic97321af6f8bbf7ad5d96e58655c342db3978a6a
Reviewed-on: https://chromium-review.googlesource.com/241944Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

Also fixes the unpack buffer offset which must be in pixel instead of
bytes in the shader parameter.

BUG=angle:508

Change-Id: I908460a76c300d96054a02206e1f2e3ad2b8e9d0
Reviewed-on: https://chromium-review.googlesource.com/241440Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Gregoire Payen de La Garanderie <Gregory.Payen@imgtec.com>

Also remove mCurPointGeometryShader and instead pass the program to
drawArrays.

BUG=angle:872

Change-Id: If55517e81d9234705eabcc0e5be7b5b492858e08
Reviewed-on: https://chromium-review.googlesource.com/241421Tested-by: Gregoire Payen de La Garanderie <Gregory.Payen@imgtec.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

Change-Id: Ic9a9f498888dba0c1986faa309f78bc71711c751
Reviewed-on: https://chromium-review.googlesource.com/241943Tested-by: Cooper Partin <coopp@microsoft.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

BUG=angle:681

Change-Id: I694073c50dccd05c3117761e446eba0d15c03293
Reviewed-on: https://chromium-review.googlesource.com/238480Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

This test flakes out on some Chromium bot configurations. It could
be that we need to redesign it to use mocks or something similar
to test reliably.

Change-Id: Ib323940d52db052e18d2352e45e029bbb1b64af0
Reviewed-on: https://chromium-review.googlesource.com/242023Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

CheckDeviceFormat with a depth stencil format gives a spurious error
when called with a non-depth-stencil format. We can check the number
of format bits before we call to reduce the error spam.

Change-Id: Ia2a6e6d756c84e13effe298240f33c8c83379267
Reviewed-on: https://chromium-review.googlesource.com/242030Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

Change-Id: Iae9ac5f8fbba68dba5e49ccda7bb7eebb05c8e9a
Reviewed-on: https://chromium-review.googlesource.com/240450Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

Add support for <<, >>, &, |, ^, and their compound assignment variants
<<=, >>=, &=, |=, ^=. Also add support for bitwise not (~).

BUG=angle:870

Change-Id: I5e6a835409589556d5d58d58078fdf505cfd8da5
Reviewed-on: https://chromium-review.googlesource.com/241850Tested-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

std::string's constructor is very expensive in debug builds.
In some applications we have seen it use 10% of the CPU.
This change makes (e)gl::Error store a pointer to an std::string,
and only create a string when necessary.

Change-Id: I78bfbda86568944681517fe32df9d1556e4ebee7
Reviewed-on: https://chromium-review.googlesource.com/240495Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

The functions are emulated on HLSL, and use the native functions on
OpenGL 4.2+ and GLES3.0. Emulation for OpenGL versions <= 4.1 is not yet
implemented.

BUG=angle:865

Change-Id: I6803a1767dacdb3dca12f13924651fd38fcacb75
Reviewed-on: https://chromium-review.googlesource.com/240961Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Olli Etuaho <oetuaho@nvidia.com>

RapidJSON gives us the ability to use JSON in our performance
tests to specific the input. The license is very permissive
and according to the author the performance is excellent.

Chromium's JSON library unfortunately seems quite tied to the
"base" project.

BUG=angle:744

Change-Id: I64e72aad1ed6fa954acd851607032cb48a461348
Reviewed-on: https://chromium-review.googlesource.com/221374Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: I83f0e5aaca4aaa84e5868afcde4f27eba011d5f1
Reviewed-on: https://chromium-review.googlesource.com/238479Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: Id901d9ae66207ac5e7d2844b8cd1f6a58c4492f6
Reviewed-on: https://chromium-review.googlesource.com/238478Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: Icdffe7eb8f975ee0966e9fb241835ac541eb39f0
Reviewed-on: https://chromium-review.googlesource.com/238477Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: Id37ed1ca617f1d2b6c1bd4a86fc8fcd28b933e8f
Reviewed-on: https://chromium-review.googlesource.com/238476Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: I2ce237281cc75fdae11b5ddb4eca52058c395e7a
Reviewed-on: https://chromium-review.googlesource.com/238475Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: I87ef6856040978a9a6045d1df20fe3117def35a3
Reviewed-on: https://chromium-review.googlesource.com/238474Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: I097d6591be4e873c9a6f210cc91e3f4eb60f036b
Reviewed-on: https://chromium-review.googlesource.com/238473Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: Idcf24556e590e07cc0b6cf640d4701353a0cda5e
Reviewed-on: https://chromium-review.googlesource.com/238472Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: I1946e01ce09d99405c318723c254fe300cc5ac53
Reviewed-on: https://chromium-review.googlesource.com/238471Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: I0c0d41fb3ff9592b08ede58a8c2ec2bd2d94a0b2
Reviewed-on: https://chromium-review.googlesource.com/238470Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:681

Change-Id: I0c8d1c04ca8eefb9b618ee6635c0643d162c0cc1
Reviewed-on: https://chromium-review.googlesource.com/240241Reviewed-by: Brandon Jones <bajones@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

Inverse is emulated in HLSL by calculating the cofactor matrix and
dividing that with the determinant. This results in the transpose of the
inverse as is required. Better performing options might exist especially
for 4x4 matrices, but this is enough for a working implementation.

BUG=angle:859

Change-Id: I5185797cc1ed86865f5f4342707abdc2977a186b
Reviewed-on: https://chromium-review.googlesource.com/240331Tested-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

Make sure that "scalar %= vector" is not approved by the compiler, and
that % and %= are only exposed on GLES3.

BUG=angle:854

Change-Id: I3ac7eb607bd86a6caa905d2bef5d397688ad3fd3
Reviewed-on: https://chromium-review.googlesource.com/241410Tested-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Gregoire Payen de La Garanderie <Gregory.Payen@imgtec.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

Change-Id: Iaf151d7521e67e8fe8bca331dabdf44d72e77ca1
Reviewed-on: https://chromium-review.googlesource.com/241420Tested-by: Gregoire Payen de La Garanderie <Gregory.Payen@imgtec.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

Change-Id: I57dba132cd08bb1b3db2cda2fe0c31b97746e1a4
Reviewed-on: https://chromium-review.googlesource.com/241380Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Nico Weber <thakis@chromium.org>

Change-Id: I409527becc9d8d2d91766a4cc5a24f96fd2b7505
Reviewed-on: https://chromium-review.googlesource.com/241070Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Austin Kinross <aukinros@microsoft.com>

We would try to create SRVs with zero mip levels, when we should
be asking for a single mip level. This would happen when the base
level is equal to the max level. See the dEQP test:

texture.mipmap.2d.max_level.linear_nearest

Change-Id: I3f231f159dbdecbf7c2e61b373bfc3545875f36e
Reviewed-on: https://chromium-review.googlesource.com/240762Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

The renderability test got flipped in 6cf8e1b9.

See dEQP test:
texture.specification.teximage2d_pbo.rgba8_snorm_2d

Note: this test still fails due to a bug in isMipmapComplete, which
has a fix in a subsequent patch.

Change-Id: Ied8ad7712bbb844132b6ac37426716780c9801c9
Reviewed-on: https://chromium-review.googlesource.com/240761Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

Change-Id: I5d1ee78fceb91c05f3925e753888b10cc81ec2b8
Reviewed-on: https://chromium-review.googlesource.com/241080Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

This method is c++11 only and caused a build breakage on Mac.

BUG=angle:773

Change-Id: Ib362b8d577aa85b9d6d2b8b87840ec5197240cf2
Reviewed-on: https://chromium-review.googlesource.com/241010Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

In OpenGL ES, it is possible to sample from level 0 of a mipmapped
texture by setting GL_TEXTURE_MIN_FILTER to GL_NEAREST.

This is possible in D3D9 and D3D11 Feature Level 10_0+ via various
methods. It's not possible in D3D11 Feature Level 9_3, though.
This change works around this restriction by creating two copies of
each texture on 9_3. The textures are identical, except one has
mipmaps and one doesn't.

The D3D11 renderer figures out which texture to use at the right
time, and keeps the textures in sync with each other as necessary.
Note: each texture is only created when it's needed. It's possible
that only one (or even neither) D3D texture will be created for a
given GL texture.

Change-Id: I3c17137b4f63b9158b3abf067ad8e5d5c49d2191
Reviewed-on: https://chromium-review.googlesource.com/234522Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

This change moves VertexBuffer11::storeVertexAttributes()'s call to
Map() outside a for-loop, eliminating unnecessary calls to Map(). Since
Map() is a relatively expensive operation (even when using NO_OVERWRITE)
this change gives a noticeable performance boost in some scenarios.

Change-Id: I320111b32f2bb9eed92efbd240206e12aaa9964e
Reviewed-on: https://chromium-review.googlesource.com/240181Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Austin Kinross <aukinros@microsoft.com>

The validation for sampler completeness had a lot of duplicated code.
The special cases have been merged into the base class by checking mTarget.

BUG=angle:681

Change-Id: I11d94c1432f8fc70a1edaaf8228bbc43c3c8fff3
Reviewed-on: https://chromium-review.googlesource.com/236932Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

Re-re-land with fixes for Linux and Android.

We were getting multiply defined symbol errors on Linux for some
stuff in the common sources. Switch to using a static lib target
for the common sources, and also sync BUILD.gn with the latest gyp.

BUG=angle:773

Change-Id: Ia2d52978a167c532e5e76cb711de589855eff44e
Reviewed-on: https://chromium-review.googlesource.com/240553Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

This header should work on all available platforms, instead of
the platform-specific intrin.h or x86intrin.h.

BUG=angle:773

Change-Id: Ia012053064838f9893aa647d872e136a47cfad1a
Reviewed-on: https://chromium-review.googlesource.com/240552Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>