| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Fixes "implicit declaration of function
_mesa_get_incomplete_framebuffer" warning.
|
|
|
|
| |
Add context.h for NEED_SECONDARY_COLOR symbol.
|
|
|
|
| |
Add context.h for FLUSH_VERTICES symbol.
|
| |
|
| |
|
|
|
|
| |
with no output
|
| |
|
| |
|
|
|
|
|
| |
Also fix up comments, so that the difference between the two passes is
clarified.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Several routines directly analyze the grf-to-mrf moves from the Gen
binary code. When it is possible, the mov is removed and the message
register is directly written in the arithmetic instruction
Also redundant mrf-to-grf moves are removed (frequently for example,
when sampling many textures with the same uv)
Code was tested with piglit, warsow and nexuiz on an Ironlake
machine. No regression was found there
Note that the optimizations are *deactivated* on Gen4 and Gen6 since I
did test them properly yet. No reason there are bugs but who knows
The optimizations are currently done in branch free programs *only*.
Considering branches is more complicated and there are actually two
paths: one for branch free programs and one for programs with branches
Also some other optimizations should be done during the emission
itself but considering that some code is shader between vertex shaders
(AOS) and pixel shaders (SOA) and that we may have branches or not, it
is pretty hard to both factorize the code and have one good set of
strategies
|
|
|
|
|
| |
Clarifies program assembly, and with a little tweak to always use
constant_map, we could cut down on constant buffer payload.
|
|
|
|
|
| |
This should be more useful for developers and for bug triaging than
just generating wrong code.
|
|
|
|
| |
Fixes glsl-vs-arrays. Bug #27388.
|
|
|
|
| |
Fixes glsl-vs-point-size.
|
|
|
|
|
|
| |
This has confused me twice now. It's a fixed width of 4 (usually a
region description of <4,4,1>), not 1. If it was 1, we'd have been
skipping all over register space.
|
| |
|
| |
|
|
|
|
| |
This supersedes http://lists.freedesktop.org/archives/mesa-dev/2010-July/001442.html.
|
|
|
|
|
|
|
|
|
|
|
|
| |
The ARL value is increments of vec4 in the register file. But
PROGRAM_TEMPORARY or PROGRAM_INPUT are stored as vec4s interleaved
between the two verts being executed (thus a vec8 each), compared to
PROGRAM_STATE_VAR being packed vec4s.
Fixes:
glsl-vs-arrays-2
glsl-vs-mov-after-deref
(without regressing glsl-vs-arrays-3)
|
| |
|
|
|
|
|
| |
The previous support was overly complicated by trying to use the same
1-OWORD message for both offsets.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Otherwise, the second half isn't written, and we end up reading back
black.
Fixes the remaining junk drawn in glsl-max-varyings, and will likely
help with a number of large real-world shaders.
|
|
|
|
|
|
| |
They go into the render cache, so while we don't care about their
contents after execution, failing to note them could cause the writes
to be flushed over important buffer contents later.
|
| |
|
|
|
|
| |
Otherwise, the subsequent read may not get the written value.
|
| |
|
|
|
|
| |
To quiet a compiler warning.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When defining mipmap level 'L' and level L-1 exists and the new level's
internalFormat matches level L-1's internalFormat, then use the same hw
format. Otherwise, do the regular ctx->Driver.ChooseTextureFormat() call.
This avoids a problem where we end up choosing different hw formats for
different mipmap levels depending on how the levels are defined (glTexImage
vs. glCopyTexImage vs. glGenerateMipmap, etc).
The root problem is the ChooseTextureFormat() implementation in some
drivers uses the user's glTexImage format/type parameters in the choosing
heuristic. Later mipmap levels might be generated with different calls
(ex: glCopyTexImage()) so we don't always have format/type info and the
driver may choose a different format.
For more background info see the July 2010 mesa-dev thread "Bug in
_mesa_meta_GenerateMipmap"
|
|
|
|
|
| |
The extension never worked, the implementation returns GLX_BAD_CONTEXT
when enabling the frame tracking.
|
|
|
|
| |
Only r200 implemented it.
|
|
|
|
|
|
|
|
| |
There was confusion on both the size of message we can send, and on
what the URB destination offset means.
The remaining problems appear to be due to spilling of regs in the
fragment shader being broken.
|
| |
|
|
|
|
|
|
| |
This cleans up some chipset dependency sprinkled around, and fixes a
potential overflow of the attribute offset array for many vertex
results.
|
| |
|
| |
|
|
|
|
|
|
| |
Noticed by Henri Verbeet on IRC.
NOTE: This is a candidate for the 7.8 branch.
|
|
|
|
|
|
|
| |
This avoids calling radeonFlush() during context destruction, when
ctx->DrawBuffer would be NULL.
NOTE: This is a candidate for the 7.8 branch.
|
|
|
|
|
|
|
| |
fixes:
https://bugs.freedesktop.org/show_bug.cgi?id=28771
NOTE: This is a candidate for the 7.8 branch.
|
| |
|
| |
|
|
|
|
| |
Despite the docs, the corresponding hardware instructions are r5xx-only.
|
|\ |
|