Commits · b75fee4e9e67546dbc80b7d33cf830e65b2ceac4 · Chen Yisong / angle

26 Feb, 2014 6 commits

Fix HLSL compiler error with else-rewriting in functions. · b75fee4e

authored Feb 26, 2014

In functions with return types where we would use if-else rewriting,
we would potentially generate a spurious HLSL error that warned of
branches with no return value in the function.

This was causing a maps regression where overlays would not draw
in Earth mode.

BUG=346463

Change-Id: Icd53023286dfa43b653fd6e261b1cdb952627706
Reviewed-on: https://chromium-review.googlesource.com/187847Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

b75fee4e

Add a new TIntermRaw node type to translator. · 19f0ff5c

authored Feb 26, 2014

This raw node stores text strings that we directly copy to the
output. This allows for more tricky substitutions that don't fit
in to the HLSL/GLSL shared parsing model.

BUG=346463

Change-Id: I83661f7db82336f3817114eec96a98bc2d395b87
Reviewed-on: https://chromium-review.googlesource.com/187846Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

19f0ff5c

Prevent overflow in the variable packer by sanity checking each variable. · 7e7143d7

authored Feb 26, 2014

BUG=346489

Change-Id: I28f5751580729a4d4d77fa6fdee0b4a6628a05de
Reviewed-on: https://chromium-review.googlesource.com/188010Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Nicolas Capens <nicolascapens@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>

7e7143d7

Fix crash on multiple calls to ShInitialize(). · df007eac

authored Feb 26, 2014

The previous fix would allow the calling app to trigger
an ASSERT if it did not first call ShFinalize. Chrome
would do this in some cases, causing a test failure.

BUG=344043

Change-Id: I33fa0c4668dac8750ca949febc7f02cc7d3d95e7
Reviewed-on: https://chromium-review.googlesource.com/187845Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

df007eac

Fix not rewriting else-if blocks. · 6ebf5a36

authored Feb 26, 2014

We would miss expanding the else-if clauses in a chain
of selection statements.

BUG=346463

Change-Id: I8d16b9f25c97b8f6e7e73ad11bf7dae8e8c57ed7
Reviewed-on: https://chromium-review.googlesource.com/187844Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

6ebf5a36

Fix long error messages breaking InfoLog::append. · b716d145

authored Feb 25, 2014

Using two calls to vsnprintf allows us to correctly determine
maximum buffer size as well as fixing garbage characters from
a missing NULL terminator.

Bug report from roland@lagoa.com.

BUG=angle:561

Change-Id: I4eb156342d9106d40783a3cf57c0c1d1889390e2
Reviewed-on: https://chromium-review.googlesource.com/187843Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

b716d145

25 Feb, 2014 1 commit

Update codereview.settings to point to gerrit. · 99125692

authored Feb 25, 2014

BUG=angle:570

Change-Id: I89dcb347d0b104219263024e419ff578f8a7aa7f
Reviewed-on: https://chromium-review.googlesource.com/187841Tested-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>

99125692

24 Feb, 2014 6 commits

Allow creating fixed-size window surfaces. · 93f01ca0

authored Jan 28, 2014

BUG=320021

Change-Id: I97cdd65ac17ee142700dbf0f363891500e967571
Reviewed-on: https://chromium-review.googlesource.com/184205Reviewed-by: John Bauman <jbauman@chromium.org>
Tested-by: John Bauman <jbauman@chromium.org>

93f01ca0

Use a stable ordering when enumerating files. · cc87b463

authored Feb 24, 2014

Output the enumerated files in sorted order when running enumerate_files.py
from gyp, to prevent unnecessary buildfile churn that makes comparing
revisions noisy.

Change-Id: Ia9f759954ea1ae66628f7b6ef3419c47fe65886e
Reviewed-on: https://chromium-review.googlesource.com/187533Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

cc87b463

Improve the enumerate_files.py script. · 0f8001b5

authored Feb 06, 2014

Cleaned up the agument parsing and added support for excluding files/folders.

BUG=angle:552

Change-Id: I0d59f83fe2e20b8bc23564ecfb1f98c08c70ca94
Reviewed-on: https://chromium-review.googlesource.com/187532Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

0f8001b5

Move the enumerate_files script to the build directory. · 823c8cd5

authored Feb 06, 2014

BUG=angle:552

Change-Id: Ie45a38bb26edfe1bf6d12b5d4918110be13acc33
Reviewed-on: https://chromium-review.googlesource.com/187531Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

823c8cd5

Fix typo in Python commit id script. · 2747b9a5

authored Feb 24, 2014

BUG=angle:529

Change-Id: Ife174e3fb4cb32342f519691d1d5d5c015cf2727
Reviewed-on: https://chromium-review.googlesource.com/187541Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

2747b9a5

Re-initialize shader translator properly after one shutdown. · 1aa7f3c1

authored Feb 18, 2014

Because of a local static, we could get stuck and never re-initialize
the translator after calling ShFinalize.

BUG=344043

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

1aa7f3c1

20 Feb, 2014 2 commits

Detect cross-process HWNDs and verify DXGI 1.2 exists in D3D11. · 836bd217

authored Feb 18, 2014

DXGI 1.2 is required to create a swap chain for a HWND owned by another
process.  This fix will allow us to fall back to creating a Renderer9 if
DXGI 1.2 is not available instead of failing to create the swap chain
later.

BUG=angle:562

Change-Id: Iad1b37967e38452cae201a6d6fa621ad66ce5f3c
Reviewed-on: https://chromium-review.googlesource.com/186977Tested-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>

836bd217

Fix git call syntax in commit_id.bat · aacedfcf

authored Feb 19, 2014

BUG=angle:566

Change-Id: I437bdec916dbfe12cb310a20d4f38c18072eacd0
Reviewed-on: https://chromium-review.googlesource.com/187210Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

aacedfcf

19 Feb, 2014 1 commit

Fix standalone Linux build. · a39b3ab9

authored Feb 14, 2014

We can include Makefiles in the ignore list, and need a few extra
flags to fix the GCC build on Linux.

BUG=angle:563

Change-Id: Iacd386359f0c0d9bb87ddc544dd8b4da2cb8fee0
Reviewed-on: https://chromium-review.googlesource.com/186902Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

a39b3ab9

18 Feb, 2014 1 commit

Fix commit header generation on non-Windows platforms. · 4df02c1e

authored Feb 14, 2014

Previously we relied on a batch file, which for obvious reasons
isn't cross-platform.

BUG=angle:529

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

4df02c1e

14 Feb, 2014 1 commit

Fix commit id versioning when building with Chrome. · 16af1e41

authored Feb 14, 2014

The GYP syntax would cause problems with the ninja build.

BUG=angle:529

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

16af1e41

13 Feb, 2014 2 commits

Tie program binary version to commit hash. · d858468f

authored Jan 10, 2014

Store the commit date in version.h, and in the DLL properties.
This will allow Chrome, or other apps who reference ANGLE internally,
to pull chronological version information about ANGLE.

BUG=angle:529
Change-Id: I307bf44103913aaaeb875a2e3ade1fab77838c48
Reviewed-on: https://chromium-review.googlesource.com/182142Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

d858468f

Refresh generated projects. · 217b9a71

authored Feb 12, 2014

Change-Id: I0764df59d1c1d327d6d1b19724eb5ee946c22ef6
Reviewed-on: https://chromium-review.googlesource.com/186124Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

217b9a71

11 Feb, 2014 2 commits

Add an option to initialize varyings without static use in vertex shaders · a25382b9

authored Feb 11, 2014

ANGLEBUG=554
TEST=webgl conformance test on mac: shaders-with-varyings.html
r=kbr@chromium.org,nicolascapens@chromium.org
cc=alokp@chromium.org,shannonwoods@chromium.org

Change-Id: I2e692d43fb15f1cf3ade3e398020d1fedb2b32f0
Reviewed-on: https://chromium-review.googlesource.com/185922Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Tested-by: Zhenyao Mo <zmo@chromium.org>

a25382b9

Draft spec for ANGLE_platform_angle_d3d · dc18dc30

authored Feb 04, 2014

This is the initial draft of a spec that extends
EGL_EXT_platform_base to allow choosing a particular ANGLE D3D
backend.

Change-Id: I1d80fa8b3c4a59fe998f6d15aa0f393bdb9d26c1
Reviewed-on: https://chromium-review.googlesource.com/184975Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
Tested-by: Scott Graham <scottmg@chromium.org>

dc18dc30

07 Feb, 2014 2 commits

Disable MRT support when using D3D10 feature levels. · aa82495c

authored Feb 07, 2014

D3D10 cards perform poorly when the pixel shader outputs to multiple
render targets that are not bound.

BUG=325581

Change-Id: I13f6f98861ffeb77cd21f7401ded4425e4ea7841
Reviewed-on: https://chromium-review.googlesource.com/185400Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

aa82495c

Fix initializing the MaxExpressionComplexity and MaxCallStackDepth compiler limits. · ef26f3bb

authored Jan 31, 2014

BUG=339860

Change-Id: Ia34304247f0aebaf00878c28f658f35a773b8061
Reviewed-on: https://chromium-review.googlesource.com/184581Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Nicolas Capens <nicolascapens@chromium.org>

ef26f3bb

31 Jan, 2014 2 commits

Fix DX11 HLSL code for texture2DProjLod. · b227fe0b

authored Jan 31, 2014

BUG=angle:549

Change-Id: I65851ad1d3ef6371e4bd6907c99e4747f9b01e81
Reviewed-on: https://chromium-review.googlesource.com/184524Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Nicolas Capens <nicolascapens@chromium.org>

b227fe0b

Add stdio.h dependency to Common.h. · 17918f9a

authored Jan 30, 2014

This missing include was also breaking the build on linux.

Change-Id: I55ba5e4df83dc8f5268d7c4afdd92e83add6aca5
Reviewed-on: https://chromium-review.googlesource.com/184376Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Commit-Queue: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

17918f9a

30 Jan, 2014 4 commits

Remove reference to CHAR_BIT from Common.h. · d02dac9f

authored Jan 30, 2014

CHAR_BIT is undefined in certain GCC builds in Linux.

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

d02dac9f

Add missing <limits> to Common.h. · ba3e2e1b

authored Jan 30, 2014

On Windows we included limits.h via map.h, and it was missing
on Linux, causing build failures in the try bots.

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

ba3e2e1b

Regenerate our prepackaged VS2010E project files. · 74efb6d0

authored Jan 30, 2014

We neglected to regenerate the project files with the addition
of RewriteElseBlocks.h/cpp.

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

74efb6d0

Fix texture formats backed by extra channels. · 0654d32a

authored Jan 27, 2014

Fix rendering to, initializing and clearing textures that are backed by more channels in D3D11 than the GL format has.

Change-Id: I250c349254acf4e6f85d4eff58e2190edb1782ef
Reviewed-on: https://chromium-review.googlesource.com/183994Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Commit-Queue: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

0654d32a

29 Jan, 2014 1 commit

Add work-around for D3D9 shader compiler bug. · 3f2c0b38

authored Jan 28, 2014

With certain selection statements with a vertex input in the
condition and side-effects in the else-block, we'd run in to
a D3D9 compiler bug which would cause incorrect results.

We can work around this bug in D3D9 by selectively rewriting
these statements to not use an else clause.

BUG=322794

Change-Id: I93c96fb201ff4959c00d9a36321faac7e0343278
Reviewed-on: https://chromium-review.googlesource.com/184021Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Commit-Queue: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

3f2c0b38

27 Jan, 2014 1 commit

Remove the ANGLE_COMPILE_OPTIMIZATION_LEVEL precompiler definition override to… · a0931907

authored Jan 27, 2014

Remove the ANGLE_COMPILE_OPTIMIZATION_LEVEL precompiler definition override to default to D3DCOMPILE_OPTIMIZATION_LEVEL3.

BUG=322795
R=geofflang@chromium.org

Change-Id: I7d8a2676c3510d26e9f0876aa30c6d8b4ae89f24
Reviewed-on: https://chromium-review.googlesource.com/183995Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Nicolas Capens <nicolascapens@google.com>

a0931907

23 Jan, 2014 1 commit

Only define ANGLE_ENABLE_PERF in libGLESv2 and libEGL. · 0af35c74

authored Jan 20, 2014

Change-Id: Ia0b8b0e67a0af5eeeec6e3fc7bfdbfdc70f519f8
Reviewed-on: https://chromium-review.googlesource.com/183161Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

0af35c74

16 Jan, 2014 2 commits

Remove __stdcall from non-windows builds. · e1590b89

authored Jan 16, 2014

Change-Id: I424cb538808ec64137c85af3bbe8742043e1c573
Reviewed-on: https://chromium-review.googlesource.com/182840Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

e1590b89

Remove system.h. · d308f3a9

authored Jan 16, 2014

The gyp files already define NOMINMAX and WIN32_LEAN_AND_MEAN.  windows.h
is also included via the D3D includes.

This ends up speeding up the build process significantly by not having
windows.h included in nearly every file.

Also fixes issues on non-windows systems by not having stdarg.h included
in debug.cpp.

Change-Id: Id32d9aac37a5a29df832a062dd8024302a798a1c
Reviewed-on: https://chromium-review.googlesource.com/182810Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

d308f3a9

15 Jan, 2014 3 commits

Rework tracing and perf defines. · 06976597

authored Oct 07, 2013

Changed ANGLE_DISABLE_* to ANGLE_ENABLE_* for perf and tracing defines so
they are disabled by default. Updated the gyp files to only turn on perf
by default for windows debug builds.

Change-Id: I71706674e6d12fbf4208acc8f100d963b82c7674
Reviewed-on: https://chromium-review.googlesource.com/182146Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

06976597

Fix incorrect include on posix systems. · 7539f75f

authored Jan 13, 2014

Change-Id: I73dbb08796c3e3a7a95f36dbf378b37eb9f936a7
Reviewed-on: https://chromium-review.googlesource.com/182145Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

7539f75f

Don't include windows.h on non-windows systems. · 2167ecdf

authored Jan 13, 2014

Change-Id: I8cf650073ead5b52bac44e31c8c3c894bc31e01c
Reviewed-on: https://chromium-review.googlesource.com/182156Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

2167ecdf

13 Jan, 2014 1 commit

Merge ES3 direct buffer changes and split index and vertex buffers. · 2c40ce44

authored Jan 08, 2014

Change-Id: I7ecdb049352af0dd94e8fbaede5953dfad9a75ef
Reviewed-on: https://chromium-review.googlesource.com/181903Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

2c40ce44

10 Jan, 2014 1 commit

Generate public projects. · 2b8b3b77

authored Dec 04, 2013

Change-Id: Id8bf2b9cf44f9fcd79eff774db64cb59b9aebd73
Reviewed-on: https://chromium-review.googlesource.com/178786Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Commit-Queue: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Nicolas Capens <nicolascapens@chromium.org>

2b8b3b77