| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Fixes: glsl-vs-varying-array
|
|
|
|
|
|
| |
Shader inputs appear in source registers, not dst registers. Catches
unsupported shaders in glsl-fs-varying-array and Humus
RaytracedShadows.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Per the GLSL 1.20 specification (presumably a clarification of 1.10).
Also, when creating user functions, make a new ir_function that shadows the
built-in ir_function, rather than adding new signatures. User functions
are supposed to hide built-ins, not overload them.
Fixes piglit tests redeclaration-{04, 12, 14}.vert.
|
|
|
|
|
|
| |
Also rename it to "is_builtin" for consistency.
Signed-off-by: Ian Romanick <[email protected]>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Moving the check for an earlier variable declaration helps cleanly
separate out the re-declaration vs. new declaration code a bit. With
that in place, conflicts between variable names and structure types or
function names aren't caught by the earlier "redeclaration" error
message, so check the return type on glsl_symbol_table::add_variable
and issue an error there. If one occurs, don't emit the initializer.
Fixes redeclaration-01.vert and redeclaration-09.vert.
Signed-off-by: Ian Romanick <[email protected]>
|
|
|
|
|
|
| |
Instead, make a new ir_function and try to add it to the symbol table.
Fixes piglit test redeclaration-08.vert.
|
|
|
|
|
|
| |
Instead, rely on the symbol table's rules.
Fixes redeclaration-02.vert.
|
|
|
|
|
|
|
|
|
|
|
|
| |
As of 1.20, variable names, function names, and structure type names all
share a single namespace, and should conflict with one another in the
same scope, or hide each other in nested scopes.
However, in 1.10, variables and functions can share the same name in the
same scope. Structure types, however, conflict with/hide both.
Fixes piglit tests redeclaration-06.vert, redeclaration-11.vert,
redeclaration-19.vert, and struct-05.vert.
|
|
|
|
| |
Fixes llvmpipe regression from one of the prev commits.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Intel sometimes uses packed depth/stencil buffers even when only a depth
buffer or only a stencil buffer was requested. Common code currently
uses the _BaseFormat field to determine whether a depth/stencil wrapper
is necessary. But unless the user explicitly requested a packed
depth/stencil buffer, the _BaseFormat field does not encode this
information, and the required wrappers are not created.
The problem was introduced by commit 45e76d2665b38b ("mesa: remove a
bunch of gl_renderbuffer fields"), which killed off the _ActualFormat
field upon which the decision to create a wrapper used to be made. This
patch changes the logic to use the Format field instead, which is more
like the old code.
Fixes fdo bug 27590.
Signed-off-by: Nick Bowler <[email protected]>
Signed-off-by: Brian Paul <[email protected]>
|
|
|
|
|
| |
Signed-off-by: Nick Bowler <[email protected]>
Signed-off-by: Brian Paul <[email protected]>
|
|
|
|
|
|
|
| |
when we dont know max_index we cannot calculate vb size from count
anymore - just use the bo size.
Also added an assert to remind that we dont handle GL_INT GL_DOUBLE
upload when we dont' know max_index - will fix later
|
|
|
|
|
|
|
|
|
|
| |
Keith prefers a clean separation between graw applications and
implementations, where apps do not link libgallium.a but instead
get all functionality they need via graw interface.
Although this is not incompatible with late loading of graw drivers, it
it would make it very hard to maintain, as wrappers for every utility
symbol exposed in graw would have to be written or generated somehow.
|
|
|
|
|
|
|
| |
stObj->pt should be set in st_bind_surface, just as in st_TexImage. On
the other hand, st_TexImage should unreference stObj->pt. It also needs
to initialize the texture image again as _mesa_clear_texture_object
clears the image.
|
| |
|
| |
|
|
|
|
|
| |
A surfaceless current context is a context that is made current without
draw and read framebuffers. Such contexts can only render to FBOs.
|
|
|
|
| |
It should be called DRM backend now.
|
|
|
|
|
|
| |
Include p_format.h for enum pipe_format symbol.
Fixes r300g build.
|
|
|
|
|
| |
Remove p_format.h.
Include p_compiler.h for boolean and uint64_t symbols.
|
|
|
|
|
|
| |
Remove p_state.h.
Include p_compiler.h for boolean symbol.
Add needed forward declarations after removing p_state.h.
|
|
|
|
| |
Include p_state.h for pipe_shader_state symbol.
|
|
|
|
| |
Include p_state.h for PIPE_MAX_COLOR_BUFS symbol.
|
|
|
|
|
| |
Include p_compiler.h for boolean symbol.
Clean up forward declarations.
|
|
|
|
| |
Include p_compiler.h for stdint.h uint*_t symbols.
|
|
|
|
|
| |
Remove p_compiler.h and p_defines.h.
Include pb_buffer.h for pb_size symbol.
|
|
|
|
|
|
| |
The previous implementation was missing handling of some rvalues, such
as "if" conditions, leading to glsl-mat-int-from-ctor-* not getting
caught.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We had ad-hoc handled some common cases by flagging sampler-typed
variables as read_only, and rejected initializers of samplers.
However, people could sneak them in in all sorts of surprising ways,
like using whole-array or structure assignment.
Fixes:
glslparsertest/glsl2/sampler-01.frag
glslparsertest/glsl2/sampler-03.frag
glslparsertest/glsl2/sampler-04.frag
glslparsertest/glsl2/sampler-06.frag
Bug #27403.
|
|
|
|
|
| |
I don't know of any problems with CPU detection in years. Don't spam
the user that was just looking to enable assertions with this stuff.
|
| |
|
| |
|
|
|
|
|
| |
Include p_compiler.h for uint symbol.
Clean up forward declarations.
|
|
|
|
| |
Include p_compiler.h for boolean symbol.
|
|
|
|
| |
Include p_state.h for pipe_surface symbol.
|
| |
|
|
|
|
|
| |
Include p_compiler.h for boolean symbol.
Include u_debug.h for assert symbol.
|
|
|
|
|
| |
Pointless now that the graw tests can be built independently of any graw
implementation.
|
|
|
|
|
|
|
|
|
| |
This allows to build multiple graws libs simultaneously and avoid
unnecessary rebuilds of the tests.
Also remove graw_util.c from inside the graw implementation -- it was
only being provided by one implementation, and graw tests were linking
against gallium anyway.
|
|
|
|
| |
Sconscript could be invoked twice if specified in the command line.
|
|
|
|
|
| |
Include p_format.h for enum pipe_format symbol.
Include p_state.h for pipe_box symbol.
|
|
|
|
|
|
| |
Remove p_defines.h.
Remove unnecessary forward declarations.
Add forward declaration for pipe_context.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 001a7bfdfc8b3c8930d5ced21982dbdfb8cd35b3. I
hadn't found the section of the spec clarifying that the old behavior
was right. Reverting fixes the new version of the testcase, and the
Humus demos that could no longer find their uniforms.
Bug #29782
Bug #29783
|
|
|
|
| |
Signed-off-by: Ian Romanick <[email protected]>
|
|
|
|
|
|
|
| |
Texcoords in AmbientApertureLighting were getting trashed since the
move of math arguments to implied moves, due to the logic for
detecting ALU message reg writes overriding the logic for SEND
implicit message reg writes.
|