Commits · d470f472b06ee11fa2fe3e6136e27d5566a85d91 · Chen Yisong / angle

21 Oct, 2015 2 commits

Don't ASSERT shader compilation success in ShaderGL. · d470f472

authored Oct 20, 2015

This assertion was useful for debugging the initial implementation but it
is becoming an issue in tryjobs where tests will hang instead of fail due
to driver bugs.

BUG=angleproject:882

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

d470f472

Re-land "Only require that the stencil masks are same as many effective bits set" · 85769f0d

authored Oct 20, 2015

It seems reasonable to validate the equality between stencilWriteMask
and stencilBackWriteMask only by comparing the effective bits.
An existing dEQP test[1] and a WebGL test[2] also use the max size of
stencil bits for checking the stencil mask values.

[1] StencilWriteMaskSeparateTestCase (gles3/functional/
es3fIntegerStateQueryTests.cpp)
[2] https://www.khronos.org/registry/webgl/sdk/tests/conformance/state/
gl-get-calls.html (Search for "minimumRequiredStencilMask ")

Re-land with fix for Clang warning.

Change-Id: I4cadaffced04ca55fa6daf7ac6a462368e0cfeec
Reviewed-on: https://chromium-review.googlesource.com/307530Reviewed-by: Jinyoung Hur <hurims@gmail.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

85769f0d

20 Oct, 2015 2 commits

Revert "Only require that the stencil masks are same as many effective bits set" · 1220bbbd

authored Oct 20, 2015

Reverting temporarily, causes a compile warning with Clang:

..\..\third_party\angle\src\libANGLE\renderer\d3d\d3d11\Renderer11.cpp(242,7) :  error: field 'mDebug' will be initialized after field 'mCurStencilSize' [-Werror,-Wreorder]
      mDebug(nullptr),
      ^
1 error generated.
ninja: build stopped: subcommand failed.

http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang_dbg/builds/98

This reverts commit 685aa4b8.

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

1220bbbd

Add support for EAC and ETC2 texture formats with D3D11. · de6144f1

authored Oct 20, 2015

Passes:
 * dEQP-GLES3.functional.implementation_limits.num_compressed_texture_formats
 * dEQP-GLES3.functional.implementation_limits.compressed_texture_formats
 * dEQP-GLES3.functional.texture.*eac*
 * dEQP-GLES3.functional.texture.*etc2*

BUG=angleproject:598

Change-Id: Iaec1756b540f8f44c821fd8b9ba63f115ebe6de0
Reviewed-on: https://chromium-review.googlesource.com/305520Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

de6144f1

19 Oct, 2015 6 commits

Fix ANGLE documentation about Chromium build. · 114e8ef2

authored Oct 19, 2015

BUG=angleproject:848

Change-Id: If70498c10d50dad3da3852c12c6bd732166c54e4
Reviewed-on: https://chromium-review.googlesource.com/307070Reviewed-by: Kenneth Russell <kbr@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

114e8ef2

Make DebugAnnotator::getStatus work on Windows 10 (WinRT) · fe7e9ed4

authored Oct 13, 2015

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

fe7e9ed4

Do not attempt to compile angle_end2end_tests or angle_perftests on Android · 27460dfc

authored Oct 15, 2015

BUG=507294

Change-Id: Ied198a26f5eecab1dc608a06c2cc5e3be08cc28c
Reviewed-on: https://chromium-review.googlesource.com/306460Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Tested-by: Peter Kotwicz <pkotwicz@chromium.org>

27460dfc

Fix MultipleDrawBuffers sample when GL_OES_texture_npot isn't active · adb857b2

authored Oct 15, 2015

If the GL_OES_texture_npot extension isn't active, then setting
GL_TEXTURE_WRAP_S/T to GL_REPEAT for a NPOT texture results in black
rendering. This is specified in Section 3.8.2 of the OpenGL ES 2.0
specification.

Setting GL_TEXTURE_WRAP_S/T to GL_CLAMP_TO_EDGE is valid even if the
NPOT extension isn't active.

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

adb857b2

Fix debug annotations and shaders. · d2c52e3b

authored Oct 14, 2015

The Program refactor inadvertantly broken Debug annotations. Fix this
by correctly passing the path to the temporary shader source in the
first "C" string argument to ShCompile, instead of as the first part
of the source string.

BUG=angleproject:1177
TEST=manual testing with MSVS

Change-Id: Ie77bb11b51645a474b542ddd2ae0f391e019e341
Reviewed-on: https://chromium-review.googlesource.com/306210
Tryjob-Request: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>

d2c52e3b

Add missing include guard to FunctionsWGL.h · 27746966

authored Oct 14, 2015

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

27746966

16 Oct, 2015 3 commits

Only require that the stencil masks are same as many effective bits set · 685aa4b8

authored Sep 29, 2015

It seems reasonable to validate the equality between stencilWriteMask
and stencilBackWriteMask only by comparing the effective bits.
An existing dEQP test[1] and a WebGL test[2] also use the max size of
stencil bits for checking the stencil mask values.

[1] StencilWriteMaskSeparateTestCase (gles3/functional/
es3fIntegerStateQueryTests.cpp)
[2] https://www.khronos.org/registry/webgl/sdk/tests/conformance/state/
gl-get-calls.html (Search for "minimumRequiredStencilMask ")

Change-Id: I68cefbe8a9f69c5a9ebcb30027a05581e3024add
Reviewed-on: https://chromium-review.googlesource.com/302703Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Tested-by: Jinyoung Hur <hurims@gmail.com>
Commit-Queue: Jinyoung Hur <hurims@gmail.com>
Tryjob-Request: Jamie Madill <jmadill@chromium.org>

685aa4b8

angle: move DeallocateCurrent into windows only code · f3586994

authored Oct 14, 2015

DeallocateCurrent is only called from within DllMain, which means
it's only called on windows.

On the mac, the warning unused-function may be triggered.

Move DeallocateCurrent to windows only code.

Change-Id: I61ba602af78d546a91cf009ae7350cf03eaa84dd
Reviewed-on: https://chromium-review.googlesource.com/305880Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Tryjob-Request: Corentin Wallez <cwallez@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

f3586994

Fixed incorrect buffer count for emulated points rendering · e115c3be

authored Oct 14, 2015

Emulated points rendering for FL9_3 was getting lucky because the
value of maxDiff just happened to be large enuough to include the
additional pointsprite vertex buffer in the total buffer count used
in IASetVertexBuffers.

maxDiff is calculated while buffers and offsets are being configured.
The pointsprite buffer was not updating maxDiff resulting in an off-by
one calculation which causes TDRs on some Qualcomm chipsets.

This issue was discovered using Cocos2dx.

BUG=angleproject:1176

Change-Id: I3b3af1757e1cef48bbc62e68695a3e239fef77bf
Reviewed-on: https://chromium-review.googlesource.com/305752Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tryjob-Request: Jamie Madill <jmadill@chromium.org>
Tested-by: Cooper Partin <coopp@microsoft.com>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>

e115c3be

15 Oct, 2015 3 commits

Don't include samplers in uniform vector count validation · 7a3e8e24

authored Oct 08, 2015

Change-Id: Id2820643b70d3266c82eb10a5f05d3a5886e9323
Reviewed-on: https://chromium-review.googlesource.com/304871Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Tryjob-Request: Jamie Madill <jmadill@chromium.org>
Tested-by: Austin Kinross <aukinros@microsoft.com>

7a3e8e24

UniformTest.SamplerUniformsAppearOnce should check for VTF support · 6655878f

authored Oct 07, 2015

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

6655878f

Fix "blob content mismatch between level9 and d3d10 shader" errors on 9_3 · d4d5084b

authored Oct 07, 2015

Fixes these dEQP failures on 9_3:
+ dEQP-GLES2.functional.shaders.struct.local.loop_nested_struct_array_vertex
+ dEQP-GLES2.functional.shaders.struct.local.loop_nested_struct_array_fragment

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

d4d5084b

14 Oct, 2015 3 commits

Re-land "Implements more pack/unpack states." · adff67b5

authored Oct 14, 2015

Pack: row length, skip rows, skip pixels.
Unpack: image height, skip images, skip rows, skip pixels.
Note that PBOs are not covered by this change.

Re-land with fix for test expectations.

BUG=angleproject:512
BUG=angleproject:1095

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

adff67b5

Revert "Implements more pack/unpack states. Pack: row length, skip rows, skip… · c7473924

authored Oct 14, 2015

Revert "Implements more pack/unpack states. Pack: row length, skip rows, skip pixels. Unpack: image height, skip images, skip rows, skip pixels. Note that PBOs are not covered by this change."

Expectations still not correct. This removes suppressions for two
failing tests:

functional.texture.specification.teximage3d_depth_pbo.depth_component32f_2d_array
dEQP-GLES3.functional.texture.specification.teximage3d_depth_pbo.depth32f_stencil8_2d_array

BUG=angleproject:512
BUG=angleproject:1095

This reverts commit 72e7013e.

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

c7473924

Automatically enable highp in fragment shaders on ESSL3 · a6996685

authored Oct 12, 2015

Most code using the translator already enables highp with the resources
flag when a shader spec that accepts ESSL3 is used, but for example the
shader_translator utility doesn't. This fix makes sure that highp is
always enabled when a fragment shader written in ESSL3 or newer is being
compiled. This will make shader_translator easier to use for testing
ESSL3 shaders.

BUG=541550
TEST=angle_unittests

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

a6996685

13 Oct, 2015 5 commits

Revert "Fixed FL9_3 shaders that use glPointCoord without glPointSize." · e73a1e84

authored Oct 13, 2015

Failing test in angle_end2end_tests:

GLSLTest.MaxMinusTwoVaryingVec4PlusTwoSpecialVariables:
program link failed: C:\fakepath(87,5-18): error X4000: variable 'output' used without having been completely initialized

This reverts commit 73e32ee1.

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

e73a1e84

Implements more pack/unpack states. · 72e7013e

authored Oct 01, 2015

Pack: row length, skip rows, skip pixels.
Unpack: image height, skip images, skip rows, skip pixels.
Note that PBOs are not covered by this change.

BUG=angleproject:512
BUG=angleproject:1095

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

72e7013e

Fixed FL9_3 shaders that use glPointCoord without glPointSize. · 73e32ee1

authored Sep 21, 2015

This change fixes a Shader error on FL9_3 that reports
Vertex/Pixel Shader linkage signatures between stages
being incompatible when glPointCoord is used without glPointSize.

Change-Id: I93ffb6c6dd38f26e2156a374b1b58ecc3a5f250b
Reviewed-on: https://chromium-review.googlesource.com/301450Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Cooper Partin <coopp@microsoft.com>

73e32ee1

Use named value-parameterized tests for dEQP gtests · 02f98029

authored Oct 13, 2015

This replace the non-descriptive digit at the end of the test name by
the name of the dEQP test being run.

Reland, enabling the named tests only in standalone.

BUG=angleproject:1153

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

02f98029

glTexSubImage2D should always allow non-power-of-two inputs · 08528e17

authored Oct 07, 2015

If GL_OES_texture_npot isn't active then ANGLE currently prevents
calls to glTexSubImage2D with:
- level > 0
- width/height NPOT

This isn't correct. It is legitimate to supply data to a NPOT subregion of a
POT texture via glTexSubImage2D.

Fixes these dEQP tests on 9_3:
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.a8_2d
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.a8_cube
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.l8_2d
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.l8_cube
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.la88_2d
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.la88_cube
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgb565_2d
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgb565_cube
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgb888_2d
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgb888_cube
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgba4444_2d
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgba4444_cube
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgba5551_2d
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgba5551_cube
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgba8888_2d
dEQP-GLES2.functional.texture.specification.basic_texsubimage2d.rgba8888_cube

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

08528e17

12 Oct, 2015 1 commit

Allow double underscore in macro names · 2f6ddf31

authored Sep 22, 2015

Double underscore is allowed according to GLSL ES 3.10, and based on
Khronos discussions the intent is that this should also apply to older
specs. The dEQP tests also check this, and WebGL tests that check the
opposite were recently removed. The error is changed into a warning.

BUG=angleproject:989
TEST=angle_unittests
     dEQP-GLES3.functional.shaders.preprocessor.* (2 tests start passing)
     dEQP-GLES2.functional.shaders.preprocessor.* (2 tests start passing)

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

2f6ddf31

09 Oct, 2015 5 commits

Made program binaries cache information needed for getTransformFeedbackVarying · 1048ea75

authored Oct 06, 2015

BUG=angleproject:1173
Change-Id: Ib8883b32f54f5d38fb94e85decae2dcee108facf
Reviewed-on: https://chromium-review.googlesource.com/304381Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Tested-by: Brandon Jones <bajones@chromium.org>

1048ea75

Treat linker warnings as errors on Windows · 2aaab29f

authored Oct 08, 2015

This would have avoided the issue seen here:
https://chromium-review.googlesource.com/#/c/304501/

Currently linker warnings are only visible in build
logs. VS outputs 0 warnings/0 errors even if there are
linker warnings.

If linker warnings were treated as errors, then the
issue above would have caused a build break.

Change-Id: I4b529b778989a7299f83e0ef3e40cccf2c17e905
Reviewed-on: https://chromium-review.googlesource.com/304886Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tryjob-Request: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

2aaab29f

Program: Clean up UBO info gathering. · 4a3c2341

authored Oct 08, 2015

The data size & block member info was getting messy, so clean up
how we query this from the Impl layer.

Also remove the register information from gl::UniformBlock, moving
it into the D3D-only world.

BUG=angleproject:1172

Change-Id: I40af658ebbd6b7c1a4251906a387ebcbb621cf77
Reviewed-on: https://chromium-review.googlesource.com/304150Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Tryjob-Request: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

4a3c2341

Revert "Fixed compilation on mingw." · e826b9f7

authored Oct 09, 2015

This causes some linker warnings. I don't think we should
add linker warnings to Chrome, so reverting this temporarily until
we can land your CL with the correct using of export keywords.

Please re-submit the fixed CL.

This reverts commit 56627322.

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

e826b9f7

Revert "Implements more pack/unpack states. Pack: row length, skip rows, skip… · 000b2f49

authored Oct 09, 2015

Revert "Implements more pack/unpack states. Pack: row length, skip rows, skip pixels. Unpack: image height, skip images, skip rows, skip pixels."

This is causing failures in dEQP-GLES3.functional.texture.specification.teximage3d_depth_pbo.depth_component24_2d_array

First failing build on the bot:
http://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20dEQP%20%28NVIDIA%29/builds/3635

Please fix the failure and re-land your CL with the fix.

BUG=angleproject:512
BUG=angleproject:1095

This reverts commit f1bb3f05.

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

000b2f49

08 Oct, 2015 4 commits

Implements more pack/unpack states. · f1bb3f05

authored Oct 01, 2015

Pack: row length, skip rows, skip pixels.
Unpack: image height, skip images, skip rows, skip pixels.

BUG=angleproject:512
BUG=angleproject:1095

Change-Id: I11e3bc05d23419b72c92b96aabd3f0bacd983626
Reviewed-on: https://chromium-review.googlesource.com/304370
Tryjob-Request: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

f1bb3f05

Suppress EGLSurfaceTest.MakeCurrentTwice on Mac chromium builds · 13473fae

authored Oct 08, 2015

This is to remove a failure on the intel trybots were ANGLE is currently
enable to create any context while this test expects creation to
succeed.

BUG=angleproject:891

Change-Id: I0450da3211fd958a17f6db0d7e2a85968d6e3a87
Reviewed-on: https://chromium-review.googlesource.com/304730
Tryjob-Request: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Corentin Wallez <cwallez@chromium.org>

13473fae

Only support ES3 if texture swizzling is available · af5756a1

authored Oct 01, 2015

On OSX 10.8 with an Intel HD 3000 we were wrongly exposing ES3 when
texture swizzling was not available.

BUG=angleproject:891

Change-Id: If71acdc1943ab2f013dc80cd28eab130c32fb201
Reviewed-on: https://chromium-review.googlesource.com/303392Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tryjob-Request: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Corentin Wallez <cwallez@chromium.org>

af5756a1

Fix WinRT compilation issue in ShaderD3D.cpp · 6d85f039

authored Oct 07, 2015

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

6d85f039

07 Oct, 2015 4 commits

Store WGL extensions in FunctionsWGL. · f22c6b13

authored Aug 31, 2015

Add helpers for checking which extensions are available instead of
discarding the extension strings after function loading.

BUG=angleproject:1144

Change-Id: Ie3d209cd5f6f116bbedaa618a59a4c2d7ddda4d6
Reviewed-on: https://chromium-review.googlesource.com/296360Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

f22c6b13

Add suppressions for several Intel failures. · 1ea9aaad

authored Oct 07, 2015

Also add a new test for UBOs to end2end_tests.

BUG=540538

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

1ea9aaad

Fix setting const qualifier on indexing expression · b3fbd867

authored Sep 30, 2015

Resubmitting now that separate fix for updating mangled names of types
has been merged. A unit test is added to make sure that the same
regression doesn't happen again.

Previously, constness of indexing expressions did not take the index
expression into account. Now constness correctly takes into account both
the base expression and the index expression.

Setting the type of expressions that index arrays is also simplified.

BUG=angleproject:1170
TEST=angle_unittests

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

b3fbd867

Invalidate mangled name of TType if the type is changed · b12531c9

authored Oct 05, 2015

Sometimes, the mangled name of a type is already generated when the type
is still being changed. For example, this could happen when the type of
A[i] is generated by copying the type of A (already mangled) and calling
clearArrayness() on the copy. Make sure that the mangled name gets
updated if the type is changed by clearing the mangled name when
something affecting it is changed.

BUG=angleproject:1170
BUG=538692
TEST=angle_unittests

Change-Id: I9dfacea653f56536e1573a6dbf60ff21da7cc5ab
Reviewed-on: https://chromium-review.googlesource.com/303846Tested-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>

b12531c9

06 Oct, 2015 2 commits

Implement QueryGL. · f0aa8429

authored Sep 29, 2015

Passes all tests in:
 * dEQP-GLES3.functional.occlusion_query
 * angle_end2end_tests

BUG=angleproject:887

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

f0aa8429

Fixed compilation on mingw. · 56627322

authored Oct 05, 2015

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

56627322