| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Fixes glsl-fs-convolution-2, which was blowing up due to the array
access insanity getting at the uniform values within the loop. Each
temporary was considered live across the whole loop.
|
| |
|
| |
|
|
|
|
| |
Fixes failure with glReadPixels(format=GL_RG)
|
|
|
|
|
|
|
|
|
|
| |
One, it was allocating increments of 1kb, but per thread scratch space
is a power of two. Two, the new FS wasn't getting total_scratch set
at all, so everyone thought they had 1kb and writes beyond 1kb would
go stomping on a neighbor thread.
With this plus the previous register spilling for the new FS,
glsl-fs-convolution-1 passes.
|
|
|
|
|
|
|
| |
g0 is used by others, and is expected to be left exactly as it was
dispatched to us. So manually move g0 into our message reg when
spilling/unspilling and update the offset in the MRF. Fixes failures
in texture sampling after having spilled a register.
|
|
|
|
|
| |
It can be tested with if (0) replaced with if (1) to force spilling for all
virtual GRFs. Some simple tests work, but large texturing tests fail.
|
|
|
|
|
|
|
|
|
| |
It's amazing this code worked. Basically, we would get lucky in
register allocation and the tests using frontfacing would happen to
allocate gl_FrontFacing storage and the instructions generating
gl_FrontFacing but pointing at another register to the same hardware
register. Noticed during register spilling debug, when suddenly they
didn't get allocatd the same storage.
|
| |
|
|
|
|
|
| |
It was used by the "neutral" tnl module that was dropped in
81ccb3e2ce708619f4c23537a237d61bdffdd35f.
|
| |
|
|
|
|
| |
Everything should be able to support 1.20 at this point.
|
|
|
|
|
|
| |
Otherwise consumers have to, and that's lame.
Signed-off-by: Adam Jackson <[email protected]>
|
|
|
|
|
| |
"Everyone else" does it this way, so follow suit. It's fewer
instructions, anyway.
|
|
|
|
|
| |
I don't think this should matter, but I'm not sure, and it's
recommended by a kernel checker in fulsim.
|
| |
|
|
|
|
|
|
| |
This should reduce the cost of generating shadow maps, for example.
No performance difference measured in nexuiz, though it does trigger
this path.
|
|
|
|
|
| |
These were for debugging in bringup. Now that relatively complicated
apps are working, they haven't helped debug anything in quite a while.
|
|
|
|
|
|
|
|
| |
This fixes hangs in some Z-writes-in-shaders tests, though other
pieces don't come out correctly.
Bug #30392: hang in fbo-fblit-d24s8. (still fails with bad color drawn
to some targets)
|
|
|
|
|
| |
Now that MESA_MINOR=10, we no longer need the extra '0' in the
version string.
|
|
|
|
| |
Signed-off-by: Thomas Hellstrom <[email protected]>
|
| |
|
| |
|
|
|
|
|
|
|
| |
rc_get_readers_normal() supplies a list of readers for a given
instruction. This function is now being used by the copy propagate
optimization and will eventually be used by most other optimization
passes as well.
|
| |
|
|
|
|
|
| |
It is possible for a single pair instruction arg to select from both an
RGB and an Alpha source.
|
| |
|
| |
|
|
|
|
| |
Fixes glean/bufferObject.
|
| |
|
| |
|
|
|
|
| |
Fixes fbo-blit and probably several other tests.
|
|
|
|
|
| |
XOR makes much more sense. Note that the previous code would have
failed for not(not(x)), but that gets optimized out.
|
| |
|
|
|
|
|
| |
Otherwise, it would try to handle arrays as structures, use
uninitialized memory, and crash.
|
|
|
|
|
|
| |
This fixes erroneous "bad format in do_row()" messages
Signed-off-by: Brian Paul <[email protected]>
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
We need to keep track of three different fragment shaders: Z-only, stencil-
only, and Z+stencil. Before, we were only keeping track of the first one
we encountered.
|
|
|
|
|
|
|
|
|
|
| |
We often use reg_null as the destination when setting up the flag
regs. However, on gen6 there aren't general implicit conversions to
destination types from src types, so the comparison to produce the
flag regs would be done on the integer result interpreted as a float.
Hilarity ensued.
Fixes 20 piglit cases.
|
| |
|
|
|
|
| |
Include compiler.h for ASSERT symbol.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously _LinkedShaders was a compact array of the linked shaders
for each shader stage. Now it is arranged such that each slot,
indexed by the MESA_SHADER_* defines, refers to a specific shader
stage. As a result, some slots will be NULL. This makes things a
little more complex in the linker, but it simplifies things in other
places.
As a side effect _NumLinkedShaders is removed.
NOTE: This may be a candidate for the 7.9 branch. If there are other
patches that get backported to 7.9 that use _LinkedShader, this patch
should be cherry picked also.
|
|
|
|
|
| |
I broke it in 06fd639c519214b6ebcbf29127b6d9ed429f8641 by only testing
2 generations of hardware :(
|
|
|
|
| |
These were left-over bits from when convolution was removed.
|
| |
|
|
|
|
| |
Also, update ir_to_mesa's "1.30 is unsupported" case to "handle" it.
|
|
|
|
| |
Hopefully this code can just go away soon.
|
|
|
|
|
|
|
|
|
|
|
| |
It is now to the point where we have no regressing piglit tests. It
also fixes Yo Frankie! and Humus DynamicBranching, probably due to the
piglit bias tests that work that didn't on the Mesa IR backend.
As a downside, performance takes about a 5-10% performance hit at the
moment (e.g. nexuiz 19.8fps -> 18.8fps), which I plan to resolve by
reintroducing 16-wide fragment shaders where possible. It is a win,
though, for fragment shaders using flow control.
|