Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(307)

Issue 6856100: Limit struct size to size of full shader constant. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 5 months ago by John Bauman
Modified:
11 years, 5 months ago
Reviewers:
kbr1, nicolas, apatrick1
CC:
angleproject-review_googlegroups.com
Base URL:
https://angleproject.googlecode.com/svn/trunk
Visibility:
Public.

Description

Limit struct size to size of full shader constant. The HLSL compiler can optimize out the last elements of a struct, which caused ANGLE to overwrite constants right after the end of the struct. Each element in a struct should be limited by the size of the entire struct. BUG= Committed: https://code.google.com/p/angleproject/source/detail?r=1394

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M src/libGLESv2/D3DConstantTable.h View 1 chunk +2 lines, -2 lines 0 comments Download
M src/libGLESv2/D3DConstantTable.cpp View 4 chunks +6 lines, -6 lines 0 comments Download

Messages

Total messages: 8
John Bauman
11 years, 5 months ago (2012-11-27 02:00:14 UTC) #1
kbr1
Any possibility of a test case for this (maybe something in the WebGL conformance suite)? ...
11 years, 5 months ago (2012-11-27 17:07:41 UTC) #2
nicolas
On 2012/11/27 17:07:41, kbr1 wrote: > Any possibility of a test case for this (maybe ...
11 years, 5 months ago (2012-11-27 17:16:16 UTC) #3
kbr1
On 2012/11/27 17:16:16, nicolas wrote: > On 2012/11/27 17:07:41, kbr1 wrote: > > Any possibility ...
11 years, 5 months ago (2012-11-27 17:22:05 UTC) #4
John Bauman
On 2012/11/27 17:22:05, kbr1 wrote: > On 2012/11/27 17:16:16, nicolas wrote: > > On 2012/11/27 ...
11 years, 5 months ago (2012-11-27 18:36:33 UTC) #5
apatrick1
Thanks for fixing this. I was tearing my hair out! Agree with Ken that a ...
11 years, 5 months ago (2012-11-27 20:11:16 UTC) #6
John Bauman
On 2012/11/27 20:11:16, apatrick1 wrote: > Thanks for fixing this. I was tearing my hair ...
11 years, 5 months ago (2012-11-28 00:22:54 UTC) #7
kbr1
11 years, 5 months ago (2012-11-28 01:29:43 UTC) #8
On 2012/11/28 00:22:54, John Bauman wrote:
> On 2012/11/27 20:11:16, apatrick1 wrote:
> > Thanks for fixing this. I was tearing my hair out! Agree with Ken that a
WebGL
> > regression test would be good. LGTM for the ANGLE change if none of the
> existing
> > WebGL conformance tests regressed.
> Ok, here's the new conformance test:
>
https://github.com/jbauman2/WebGL/commit/a2456978aee51ecc93885d6831a0e6a18c58...

Awesome. Thanks very much.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b