Commits · 56cf9af282aa514b1b0f29dc348ffb375ca295db · Chen Yisong / angle

17 Feb, 2015 6 commits

Add a FunctionsGL class for loading GL entry points. · 56cf9af2

authored Feb 17, 2015

BUG=angle:879

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

56cf9af2

D3D: Re-init static buffer after invalidate. · 5b8b1087

authored Feb 13, 2015

This allows the Buffer class to always know if it's intended to
use static data vs dynamic. In some cases we would end up with
NULL pointers and not know if we started with static or dynamic
usage.

BUG=angle:912

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

5b8b1087

Generalize NativeBuffer11::setData. · 026b4188

authored Feb 13, 2015

We can replace this method with a base class method which uses our
implementation of "map". This allows us to reuse setData in other
mappable buffer storage types.

BUG=angle:912

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

026b4188

Revert "Resubmit "Compile the D3D11 VS and PS on separate threads at GL link time"" · d2d21dca

authored Feb 17, 2015

Causes errors in Clang-Win. We can fix the error and then resubmit, but need to do a roll now.

This reverts commit 31018486.

Change-Id: I1c91b0a97031df33c2261089f6b54ccd3270306b
Reviewed-on: https://chromium-review.googlesource.com/250430Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

d2d21dca

Implement missing variants of min, max and clamp · 42077bb1

authored Feb 16, 2015

BUG=angle:914

TEST=dEQP tests
Tests passing 100% because of this change:
  dEQP-GLES3.functional.shaders.operator.common_functions.min.*
  dEQP-GLES3.functional.shaders.operator.common_functions.max.*
  dEQP-GLES3.functional.shaders.operator.common_functions.clamp.*
Reduced number of errors in tests:
  EQP-GLES3.functional.shaders.constant_expressions.builtin_functions.common.*
  These tests are not yet passing because of constant expressions
  bug (angle bug:913)

Change-Id: Ibccee47ed78e994e40a0effe9afa9cf9692492b0
Reviewed-on: https://chromium-review.googlesource.com/250330Reviewed-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Nicolas Capens <capn@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

42077bb1

Fixed variable shadowing warnings · f7b10164

authored Feb 12, 2015

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

f7b10164

16 Feb, 2015 2 commits

Make Buffer11.cpp build with clang. · d8d58301

authored Feb 13, 2015

Change-Id: If72bc38dbb484e56a0a26c3aa7ac38bbc319653a
Reviewed-on: https://chromium-review.googlesource.com/249921Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

d8d58301

Separate integer and float modulus · ff805cc3

authored Feb 13, 2015

Integer and float modulus are separated into Mod and IMod operator types
to make the code clearer and to make GetOperatorString return the correct
value for both.

Change-Id: Ibfbca2c558bf919b0eab6404d7349f87fe47a18c
Reviewed-on: https://chromium-review.googlesource.com/249692Tested-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>

ff805cc3

13 Feb, 2015 6 commits

Enable the OpenGL/WGL backend on Windows. · 23dcabad

authored Feb 13, 2015

BUG=angle:890

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

23dcabad

Add the OpenGL renderer to the GN build. · c588ac73

authored Feb 13, 2015

BUG=angle:890

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

c588ac73

Fixed compressed texture support reporting for feature level 9_3 · d28f6113

authored Feb 12, 2015

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

d28f6113

Remove the dependency on OpenGL32.lib by loading all WGL functions. · 12892c91

authored Feb 13, 2015

BUG=angle:879

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

12892c91

Fix Buffer11 staging buffer revision update. · d12971af

authored Feb 12, 2015

This bug could trigger a potential extra copy when drawing with
a pack buffer (or possibly packing vertex/index data).

BUG=angle:912

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

d12971af

Add a basic WGL implementation. · 398d8e6e

authored Jan 22, 2015

BUG=angle:890

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

398d8e6e

12 Feb, 2015 13 commits

Add addBinaryMath and addUnaryMath helpers to ParseContext · 09b22474

authored Feb 11, 2015

This refactoring reduces code duplication and moves functionality that is
not immediately related to the language grammar out of glslang.y.

BUG=angle:911

Change-Id: If5e225461890ed542dee01905829df1c9a6f5e27
Reviewed-on: https://chromium-review.googlesource.com/248570Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

09b22474

Disable warning 4530 in the libANGLE GN project · 40853472

authored Feb 12, 2015

Change-Id: I56b5428f40a294248305dff168b9b0bb7b0a9bcc
Reviewed-on: https://chromium-review.googlesource.com/249240Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

40853472

Clean up the style in Buffer11.cpp. · f03bcad6

authored Feb 11, 2015

Fix indentation, and simplify the naming scheme for the storage.
Also add some missing DISALLOW_COPY_AND_ASSIGN macros.

BUG=angle:912

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

f03bcad6

Move operator definitions to their own file · 1033e1d8

authored Feb 12, 2015

This generalizes GetOperatorString so that it can be used outside
ValidateLimitations.

BUG=angle:911

Change-Id: I43a160b3d7d60388ad529a72f1720d3171f67315
Reviewed-on: https://chromium-review.googlesource.com/249073Tested-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>

1033e1d8

Update WebGL failures list. · 10617310

authored Feb 11, 2015

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

10617310

Add GL and WGL renderer stubs. · f9a6f084

authored Jan 22, 2015

BUG=angle:890

Change-Id: I64f2a72b4a350f95acc2ca7080fea1a308422ca4
Reviewed-on: https://chromium-review.googlesource.com/242573Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

f9a6f084

Store mShareHandle in SurfaceD3D. · ba887d90

authored Jan 20, 2015

The type of the share handle depends on the implementation.

BUG=angle:658

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

ba887d90

Move surface size information out of SurfaceImpl. · 45b4e74a

authored Jan 20, 2015

Some implementations need to query size information directly from their
native window instead of listening for size events and storing the result.

Also move mSwapInterval to SurfaceD3D since it doesn't need to be stored
in all cases and there is no query for it.  It no longer requires that the
SurfaceImpl's derivation remembers to set mSwapInterval in
setSwapInterval too.

BUG=angle:658

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

45b4e74a

Split up the Display::create*Surface and have them match the API. · f6bf23fd

authored Jan 20, 2015

BUG=angle:658

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

f6bf23fd

Fix HLSL varying struct linking. · 9b82084f

authored Feb 12, 2015

We would always expand a struct name to always include its symbol
ID. This fixes the workaround to only affect scoped structs.
We can leave global structs, which are by definition uniquely named,
without decorating them with a symbol ID.

This fixes several tests in dEQP's shader.linkage.varying.struct.

Re-land with GLSL translator bug fixed.

BUG=angle:910

Change-Id: I23a932bd1dadea5e9aafabde697e6a2af9a43f2b
Reviewed-on: https://chromium-review.googlesource.com/249134Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

9b82084f

Fix UNREACHABLE being hit in Renderer9::applyTransformFeedbackBuffers. · b69a0923

authored Feb 12, 2015

Since applyTransformFeedbackBuffers is called even if transform feedback
is not active now, remove the UNREACHABLE.

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

b69a0923

Revert "Allow varying structs to match between stages." · b960cc4a

authored Feb 12, 2015

This patch has a bug which breaks the GLSL to GLSL translator.

See broken GPU FYI bot builds on the Chromium waterfall:

http://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20%28NVIDIA%29/builds/13074

This reverts commit 489ffc4d.

Change-Id: I8241c6d658f4f31af91b89af1141d3d778ff7c70
Reviewed-on: https://chromium-review.googlesource.com/249131Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

b960cc4a

Unbind unused stream out from the D3D11 pipeline to avoid conflicts. · 5274202f

authored Feb 04, 2015

Also allow points drawing without gl_PointSize if transform feedback is
active.

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

5274202f

11 Feb, 2015 9 commits

Use correct non-square register count in structs. · f4780c1c

authored Feb 11, 2015

For struct varyings with nested structs, we would count registers
incorrectly, and produce D3D link errors.

This fixes tests in the dEQP's shaders.linkage.varying.struct.

BUG=angle:910

Change-Id: I47ea6dba36bf57cf94a7e7f30997c6c9096c2b40
Reviewed-on: https://chromium-review.googlesource.com/247243Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>

f4780c1c

Allow varying structs to match between stages. · 489ffc4d

authored Feb 11, 2015

We would always expand a struct name to always include its symbol
ID. This workaround fixed scoped structs with the same name. It
would also block any possiblity of linking the structs successfully.

Instead we can use the workaround only on inner-scoped structs, and
leave global structs, which are by definition uniquely named, as
they are.

This fixes several tests in dEQP's shader.linkage.varying.struct.

BUG=angle:910

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

489ffc4d

Fix struct uniform packing. · 2857f489

authored Feb 09, 2015

Our current code would not adapt properly for members that did
not fit in the previous register. Instead, use the correct adjusted
value as returned from the block layout encoder.

This fixes a dEQP WebGL test: gles2/shaders/linkage.

BUG=angle:910

Change-Id: Id77d0c76ce767665b3db97cb4d14e8008254fd9f
Reviewed-on: https://chromium-review.googlesource.com/247241Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

2857f489

Implement equality ops for nested structs. · 55e79e09

authored Feb 09, 2015

This fixes the WebGL test glsl_misc_struct_equals as well as
several dEQP tests in functional.shaders.struct.

BUG=391957
BUG=angle:910

Change-Id: I09f3cd3f51bbc3541b64dbcfddfe01884ddba6f5
Reviewed-on: https://chromium-review.googlesource.com/247083Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

55e79e09

Return egl::Error from Renderer::initialize. · b2a5e342

authored Feb 11, 2015

This extra Error info can be useful for passing back additional info.
In the short-term this means we can implement a UMA histogram at some
point with a code that represents a more specific reason why the
init failed.

BUG=436191

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

b2a5e342

Cache sampler completeness in the Texture class. · 0fe40534

authored Feb 10, 2015

BUG=angle:909

Change-Id: I14e06e01d111e9d5eec415f4c2d831b61dcb1e3d
Reviewed-on: https://chromium-review.googlesource.com/248070Reviewed-by: Nicolas Capens <capn@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

0fe40534

Add Display getters for native display and attribs. · 38832d17

authored Feb 09, 2015

This simplifies the logic in the Renderer init.

BUG=angle:905

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

38832d17

Reduce the calls to get[Width|Height|InternalFormat] in Texture.cpp. · b0eecdfd

authored Feb 10, 2015

BUG=angle:909

Change-Id: I8a66c1d317bcb7a01d79f9475390deb292e1ae2e
Reviewed-on: https://chromium-review.googlesource.com/248052Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Nicolas Capens <capn@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

b0eecdfd

Store the texture image information in a std::vector instead of a std::map. · 866dd2d1

authored Feb 10, 2015

BUG=angle:909

Change-Id: Ib595306f6cf0633b16bff6389373c2bb2cc29ec9
Reviewed-on: https://chromium-review.googlesource.com/248051Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Nicolas Capens <capn@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

866dd2d1

09 Feb, 2015 4 commits

Implement eglQueryContext · 83107970

authored Feb 05, 2015

BUG=angle:895

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

83107970

Fix cstdint include in MemoryBuffer.h. · 959ad9a8

authored Feb 09, 2015

This syntax isn't compatible with OSX.

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

959ad9a8

Move rx::MemoryBuffer to common library. · 74ba76c4

authored Feb 09, 2015

This class is not D3D-specific so can be shared among other
back-ends, or other parts of the ANGLE codebase.

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

74ba76c4

Add a script to generate a new Renderer. · 811b6352

authored Feb 09, 2015

Also refactor our current Impl headers slightly to facilitate
parsing by the generation script.

BUG=angle:905

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

811b6352