| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
|
|
|
|
|
| |
Otherwise, with repeated program recompile, we never free the results
of the previous compile.
|
| |
|
|
|
|
|
| |
Fixes glslparsertest/glsl2/scoping-01.frag (successfully compiled but
should've failed) and scoping-02.frag (assertion triggered).
|
| |
|
|
|
|
|
|
|
|
| |
This should make it easier to diff the output, clean up some of the
insane whitespace, and make the strings a bit smaller.
We'll probably need to split up the prototype strings eventually, but
for now, this gets it under the 65K mark.
|
| |
|
|
|
|
|
|
| |
Calls to equal(bvec, bvec) or notEqual(bvec, bvec) previously caused an
assertion. Fixes piglit tests glsl-const-builtin-equal-bool and
glsl-const-builtin-notEqual-bool.
|
| |
|
|
|
|
| |
Doesn't fix anything, as those indices were both always 0.
|
| |
|
| |
|
|
|
|
| |
Fixes fd.o bug #29629.
|
|
|
|
| |
After a recent change to glcpp-parse.y (adding "redefined macro" error).
|
|
|
|
|
|
|
|
|
| |
Carefully avoiding printing any error when the new definition matches
the existing definition.
This fixes the recently-added 088-redefine-macro-legitimate.c and
089-redefine-macro-error.c tests as well as glsparsertest/preprocess1
in piglit.
|
|
|
|
|
|
| |
The specification says that redefining a macro is an error, unless the
new definitions is identical to the old one, (identical replacement
lists but ignoring differing amounts of whitespace).
|
|
|
|
| |
This is useful for debugging the preprocessor.
|
|
|
|
| |
the first two are straight op2's and the DDX/DDY are taken from r600c.
|
|
|
|
| |
fixes fp-set-01 and glsl-fs-step
|
|
|
|
| |
fixes biased texturing tests
|
|
|
|
|
|
| |
Don't do perspective for TEX, and also copy input to a temporary for TEX
also add tex opcode names
|
|
|
|
|
|
| |
In commit 6be3a8b70af4ba4fa4d037d54ecf6d5f055edbc9, the #version directive
was fixed to stop generating a spurious newline. Here we simply update
the expected result for the single test which includes a #version directive.
|
|
|
|
|
| |
The previous commit changed glcpp-lex.l so we commit the resulting
generated file here.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Matching the newline here meant having to do some redundant work here,
(incrementing line number, resetting column number, and returning a
NEWLINE token), that could otherwise simply be left to the existing rule
which matches a newline.
Worse, when the comment rule matches the newline as well, the parser
can lookahead and see a token for something that should actually be skipped.
For example, in a case like this:
#if 0 // comment here
fail
#else
win
#endif
Both fail and win appear in the output, (not that the condition is being
evaluated incorrectly---merely that one token after the comment's newline
was being lexed/parse regardless of the condition).
This commit fixes the above test case, (which is also remarkably similar
to 087-if-comments which now passes).
|
|
|
|
| |
makes glsl-fs-log2 and glsl1-integer division with uniform var pass
|
|
|
|
| |
fixes piglit pointAtten and point-sprite tests
|
|
|
|
|
|
|
|
|
| |
I didn't expect that this would really work, but it turns out there
are shaders in the wild that do it.
Fixes: (with swrast)
glsl-fs-main-return
glsl-vs-main-return
|
|
|
|
|
| |
We'll need to use the HALT instruction to do this right, like returns
from other functions.
|
|
|
|
| |
This still needs work, passes tex3d, fbo-scissor-bitmap, scissor-bitmap
|
|
|
|
|
| |
We were putting the source pointer in a register used as a temporary,
breaking all paths that don't read the data in a single instruction.
|
| |
|
|
|
|
|
|
|
| |
h needs to be rounded up, this probably needs revisiting when we get
to tiling etc.
fixes fbo-generatemipmap-npot
|
|
|
|
|
|
|
| |
8 more piglit tests pass,
fbo-clearmipmap, fbo-copyteximage, fbo-generatemipmap,
fbo-generatemipmap-nonsquare, fbo-generatemipmap-scissor,
fbo-generatemipmap-viewport, gen-teximage, gen-texsubimage
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These binops are the vector-to-bool comparisons, not vec-to-bvec. We
likely want both operations avilable as expression, since 915 and 965
FS naturally does the vector version, while 965 VS can also naturally
do the scalar version. However, we can save that until later.
Fixes:
glsl-fs-vec4-operator-equal.shader_test
glsl-fs-vec4-operator-notequal.shader_test
glsl-vs-vec4-operator-equal.shader_test
glsl-vs-vec4-operator-notequal.shader_test
|
|
|
|
| |
Fixes glsl-fs-sign and many of the tests of trig builtins.
|
|
|
|
|
| |
Fixes glsl-unused-varying and many others, since we produce an output
reg read any time gl_FragColor is written inside an if statement.
|
|
|
|
| |
Fixes glsl-fs-dot-vec2.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now that we have glsl2 with if flattening in place, most shaders will
just work. Remaining failing shaders will mostly be due to loop
unrolling (in progress), some possible if flattening failures in
inlining functions (planning on fixing), and the register/instruction
count limits.
While the GLSL and GLSL-ES specs say that shaders shouldn't fail to
compile/link due to register/instruction limits, in practice we're not
the first vendor to expose GLSL on hardware with these limitations.
The benefit to application developers of providing a better language
for GPU programming is greater than the pain of having to handle
instruction limits (which they had to for ARB_fp on this hardware
anyway)
|
|
|
|
|
|
|
|
|
| |
This error led to an assertion failure for some constructors of
non-square matrices. It only occured in matrices where the number of
columns was greater than the number of rows. It didn't even always
occur on those.
Fixes piglit glslparsertest case constructor-16.vert.
|
|
|
|
| |
with glsl2, reads from outputs are legal
|
|
|
|
| |
Fixes piglit test case glsl-vs-ff-frag and bugzilla #29623.
|
|
|
|
|
|
| |
In one optimization pass, register files may have been messed therefore
merging instructions which use the same index in two different register
files.
|
|
|
|
| |
Signed-off-by: Jerome Glisse <[email protected]>
|
|
|
|
| |
Fixes glsl-vs-dot-vec2.
|
|
|
|
|
|
|
|
|
|
|
|
| |
When simplifying (vec4(1.0) / (float(x))) to rcp(float(x)), we forgot
to produce a vec4, angering ir_validate when starting alien-arena.
Fixes:
glsl-algebraic-add-zero-2
glsl-algebraic-div-one-2
glsl-algebraic-mul-one-2
glsl-algebraic-sub-zero-3
glsl-algebraic-rcp-sqrt-2
|
|
|
|
|
| |
This could reduce the amount of memory used by a shader tree after
optimization, and increases consistency with other passes.
|
|
|
|
|
|
|
|
|
| |
Several optimization paths, including constant folding, can lead to
accessing an ir_constant array with an out of bounds index. The GLSL
spec lets us produce "undefined" results, but it does not let us
crash.
Fixes piglit test case glsl-array-bounds-01 and glsl-array-bounds-03.
|
|
|
|
|
| |
This will make extracting source to produce minimal testcases for
shader compile issues easier.
|
| |
|
|
|
|
|
| |
Use DRI_LIB_DEPS when linking GL/GLES state trackers. This fixes
missing talloc symbol errors, and is hopefully more future proof.
|