st/glx: added some comments

nvfx: implement LIT in fp

glsl2: Use as_constant some places instead of constant_expression_value

The places where constant_expression_value are still used in loop
analysis are places where a new expression tree is created and
constant folding won't have happened.  This is used, for example, when
we try to determine the maximal loop iteration count.

Based on review comments by Eric.  "...rely on constant folding to
have done its job, instead of going all through the subtree again when
it wasn't a constant."

glsl2: Allow copy / constant propagation into array indices

glsl2: Add module to perform simple loop unrolling

glsl2: Track the number of ir_loop_jump instructions that are in a loop

ir_expression: Add static operator_string method

I've used this in quite a few debug commits that never reached an
up-stream tree.

exec_node: Add insert_before that inserts an entire list

glsl2: Eliminate zero-iteration loops

glsl2: Perform initial bits of loop analysis during compilation

glsl2: Add module to suss out loop control variables from loop analysis data

This is the next step on the road to loop unrolling

glsl2: Add module to analyze variables used in loops

This is the first step eventually leading to loop unrolling.

ir_to_mesa: Handle loops with loop controls set

The downside of our talloc usage is that we can't really make static
(i.e., not created with new) instances of our IR types.  This leads to
a lot of unnecessary dynamic allocation in this patch.

ir_validate: Validate loop control fields in ir_loop

glsl2: Add cmp field to ir_loop

This reprents the type of comparison between the loop induction
variable and the loop termination value.

glsl2: Set a flag when visiting the assignee of an assignment

exec_list: Add pop_head

ir_print_visitor: Print empty else blocks more compactly

nvfx: fix division by zero in vp-ignore-input

nvfx: report correct max lodbias

Fixes piglit lodbias

nvfx: remove message

nvfx: support indirect addressing in vps

Negative or huge offsets not yet supported.

r600c: add proper returns for some evergreen functions

these weren't checked anyway.

Fixes:
https://bugs.freedesktop.org/show_bug.cgi?id=29999

nvfx: fix support for more than 8 texture units (fixes etqw crash)

r600g: fix segfault in state after rework

probably can improve this a bit.

r600c: emit DB_HTILE_DATA_BASE on evergreen

Make the hw happy.

r600g: refactor sample states into a reusable struct.

I will not cut-n-paste.
I will not cut-n-paste.
I will not cut-n-paste.

r600g: reduce size of r600 context structure to !insane

Its now about 7.8k, and might actually fit in a cache.

r600g: add texture border state.

Okay I finally wrapped my head around what r600_context_state is meant to be,
maybe I should just rename all the structs so that have distinct names.

I've no idea however why 16 is a good magic number for R600_MAX_RSTATE.

r600g: deref old driver states for set entry points.

r600g: drop r600_bind_state.

This was another ugly function that really wasn't needed.

The 3 calls to it from the gallium api were shorter than it,
and all the calls from the set_ functions were pointless.

r600g: kill r600_context_state function

having some sort of locality of code really matters, just create
and setup state at time. Not sure if this is just further polishing of a bad thing,
but at least it makes it more readable.

r600g: move lots of state inline helpers to separate header.

this gets them out of sight of the main codeflow.

draw: Include missing headers in draw_vs_aos.h.

Include tgsi_exec.h for TGSI_EXEC_NUM_TEMPS.
Include draw_vs.h for draw_vs_varient.

r600g: drop magic numbers in depth state.

this also fixes occulsion queries.

util: Include missing header in u_linear.h.

Include p_compiler.h for size_t and boolean symbols.

mesa: Fix printf-like warning.

mesa: Fix printf-like warnings.

r600g: force unbind of previously bind sampler/sampler_view

Previously bind sampler/sampler_view can be converted and endup
overwritting the current state we want to schedule. Example :
bind texA texB to sampler_view[0] & sampler_view[1], render,
bind texB to sampler_view[0] render. Now state associated to
texB are set to configure sampler_view slot 0, but as we don't
unbind sampler_view[1] still point to texB state so we end up
with sampler_view[1] overwritting sampler_view[0], which gives
wrong rendering if next rendering bind texA to sampler_view[0],
it will endup as texB is bound to sampler_view[0]. If you are
not confuse at that point give me a call i will be buying you
beer.

Signed-off-by: Jerome Glisse <jglisse@redhat.com>

st/mesa: clamp gl_constants::MaxVarying against MAX_VARYING

Don't try to use more generic varying vars than core Mesa supports.
Fixes fd.o bug 29959.

glsl2: Update TODO file

r600g: fix memory/bo leak

Signed-off-by: Jerome Glisse <jglisse@redhat.com>

mesa: fix code generation for ir_unop_sqrt

The CMP instruction needed to be flipped to properly handle
operand==0.

Fixes fd.o bug 29923.

mesa: fix some printf warnings with casts

r600g: fix thinko in shadow code.

spotted by taiu on irc

r600g: fix logicop, the 3d ROP is the 2D rop shifted twice.

r600g: fix depth texture tests

glsl: Apply implicit conversions to structure constructor parameters.

The code for handling implicit conversions should probably get
refactored, but for now, this is easy.

Fixes piglit test constructor-26.vert.

glsl: Convert constant record constructor parameters to ir_constants.

I'm not sure if this is strictly necessary, but it seems wise.

glsl: Reject structure constructors that have too many arguments.

Fixes piglit test constructor-27.vert.

glsl2: Remove unnecessary glsl_symbol_table::get_function parameter return_constructors

Now that constructors are not generated as functions or stored in the
symbol table, there is no need to flag whether or not constructors
should be returned.

glsl2: Remove unused method glsl_type::generate_constructor

glsl2: Remove unused 'constructor' parameter from glsl_symbol_table::add_type

glsl2: Don't generate constructor functions for structures

glsl2: Emit structure constructors inline

Fixes piglit test cases glsl-[fv]s-all-0[12].

r600g: add missing vertex fetch formats to the translation table.

fixes at least 2 more piglits.

r600g: fix binding of same texture to several target slot

One can bind same texture or sampler to different slot,
each slot needs it own state. The solution implemented
here is not exactly beautifull or optimal need to think
to somethings better.

Signed-off-by: Jerome Glisse <jglisse@redhat.com>

glsl: Fix write mask in matrix-from-matrix constructors.

If the matrix being constructed was larger than the source matrix, it
would overwrite the lower-right part of the matrix with the wrong
values, rather than leaving it as the identity matrix.

For example, constructing a mat4 from a mat2 should only use a writemask
of "xy" when copying from the source, but was using "xyzw".

Fixes the code generated by piglit test constructor-23.vert.

glsl: Add proper handling for constant matrix-from-matrix constructors.

Fixes piglit test case constructor-21.vert and changes
constructor-22.vert to give the correct output.

glsl: Move generate_constructor_(matrix|vector) to ir_constant ctor.

ast_function: Fix check for "too few components".

This was triggering even for matrix-from-matrix constructors.  It is
perfectly legal to construct a mat3 from a mat2 - the rest will be
filled in by the identity matrix.

Changes piglit test constructor-23.vert from FAIL to PASS, but the
generated code is incorrect.

ast_function: Remove bogus cases from generate_constructor_matrix.

There are no integer matrix types, so switching on them is silly.

r600g: fix incorrect state naming in pipe_sampler vs pipe_sampler_view

fixes problems in valgrind with uninitialised values.

ir_to_mesa: Load all the STATE_VAR elements of a builtin uniform to a temp.

Like the constant handling and the handling of other uniforms, we add
the whole thing to the Parameters, avoiding messy, incomplete logic
for adding just the elements of a builting uniform that get used.
This means that a driver that relies only on ParameterValues[] for its
parameters will have an increased parameter load, but drivers
generally don't do that (since they have other params they need to
handle, too).

Fixes glsl-fs-statevar-call (testcase for Ember).  Bug #29687.

Regresses glsl-vs-array-04 on 965.  Thanks to a slight change in
register allocation, this test of undefined behavior now wraps around
the register space and unexpectedly reads the constant value it's
trying to compare to.  The test should probably not look at the
resulting color, since behavior is undefined.

ir_to_mesa: Add a little helper for emitting link failure messages.

mesa: Fix many printf-like warnings.

Most of these are just typecasting to long to match the arg type.  I
don't really care too much about getting a GLsizei or whatever
appropriate type in.  However, there were a number of real bugs, like
missing arguments or passing floats to integer format specifiers.  My
favorite: printflike("%s, argument") is missing an argument.

mesa: Add __printf__ attribute to printf-like functions to get warnings.

mesa/st: remove check for buffer/elements = 0

shown by the glsl-vs-point-size failing on r600g.

the test passes on softpipe and I get a full piglit test run completing on r600g.

r600g: silence compiler warning

Signed-off-by: Jerome Glisse <jglisse@redhat.com>

r600g: refix db/cb state

Signed-off-by: Dave Airlie <airlied@redhat.com>
Signed-off-by: Jerome Glisse <jglisse@redhat.com>

mesa: fix out of bounds memory read in mipmap gen code

Out of bounds reads could happen for reducing WxH to WxH/2
or WxH to W/2xH.

Fixes fd.o bug 29918.

mesa: more prog_execute.c debug code

glsl: Add forgotten implementations of equal/notEqual on bvecs.

r600g: fix up default state differences between r6xx and r7xx

Signed-off-by: Alex Deucher <alexdeucher@gmail.com>

st/glx: re-order destruction of buffers, visuals

Free the buffers before the visuals.  Fixes valgrind warning
reported in fd.o bug 29919.

i965: DP2 produces a scalar result like DP3, DP4, etc.

Fixes glsl-fs-dot-vec2-2.

glsl2: Perform algebraic simplifications on logical binary operators

Reduces glsl-vs-all-01 from 42 Mesa IR instructions (including the
END) to 17.

glsl2: Allow ir_constant::zero to create boolean constants

r600g: avoid dynamic allocation of states

Make state statically allocated, this kills a bunch of code
and avoid intensive use of malloc/free. There is still a lot
of useless duplicate function wrapping that can be kill. This
doesn't improve yet performance, needs to avoid memcpy states
in radeon_ctx_set_draw and to avoid rebuilding vs_resources,
dsa, scissor, cb_cntl, ... states at each draw command.

Signed-off-by: Jerome Glisse <jglisse@redhat.com>

Revert "Revert "r600g: precompute some of the hw state""

This reverts commit 1fa7245c348cb7aced81f1672140f64cb6450e2f.

Conflicts:

src/gallium/drivers/r600/r600_state.c

nouveau/nvfx: Remove enforcement of bit depth being same as front buffer

Signed-off-by: Patrice Mandin <patmandin@gmail.com>

mesa: initialize dummy framebuffer and renderbuffer mutexes

See fd.o bug 29909.

Signed-off-by: Brian Paul <brianp@vmware.com>

Cygwin: Adjust mklib so -linker and -cplusplus options are processed more like they are for linux

It looks like we were ignoring -linker when -noprefix wasn't present, and
when -noprefix was present, -linker was mandatory and -cplusplus ignored.

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Signed-off-by: Brian Paul <brianp@vmware.com>

Cygwin: Teach mklib/minstall to properly install libraries on cygwin

Teach mklib/minstall more about cygwin so libraries are properly installed

Have mklib install the .dll into the lib/ staging directory as well

Have minstall install the .dll into PREFIX/bin at the same time as
installing the .dll.a link library into PREFIX/lib

mklib uses a '-' rather than a '.' as the separator before the version
number in library names on cygwin.  Change the install globs so they match
library names like that.

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Signed-off-by: Brian Paul <brianp@vmware.com>

Cygwin: Change mklib not to report the full archname on cygwin

Change mklib not to report the full archname when building a library for cygwin
(which is something like 'CYGWIN_NT-5.1' or 'CYGWIN_NT-6.1-WOW64' and kind of
confusing), but just 'CYGWIN'.

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Signed-off-by: Brian Paul <brianp@vmware.com>

Cygwin: Have mklib exit with error code if link fails

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Signed-off-by: Brian Paul <brianp@vmware.com>

glsl2: Disallow function declarations within function definitions in GLSL 1.20

The GLSL 1.20 spec specifically disallows this, but it was allowed in
GLSL 1.10.

Fixes piglit test cases local-function-0[13].frag and bugzilla #29921.

r600: cube mipmap levels are aligned to 8 faces only starting from r7xx

r600g: correct cb/zb offset emits.

This fixes fbo-3d and fbo-cubemap

Revert "r600g: precompute some of the hw state"

This reverts commit de0b76cab22caa9fc7260f80acb8f151ccced6c5, its pre-computes the texture state wrong,

you can't just use an array of levels, since you can have FBOs to depth texture slices inside a level as well
it would get really messy quickly. Probably need to split commits like this up into pieces for each piece
of state, so we can revert bits easier in case of regressions.

This also break 5 piglit tests, and valgrind starts to warn about invalid read/writes after this.

r600g: fix typo causing segfault.

fixes warning that
r600_blit.c: In function ‘r600_resource_copy_region’:
r600_blit.c:136: warning: passing argument 1 of ‘util_resource_copy_region’ from incompatible pointer type

and also 7 more piglit tests.

r600g: fix glean texCube and shadows.

add cube and shadow support to the texture code.

gallivm: fix bug in nested conditionals

This, plus the previous commit fix fd.o bug 29806.

glsl2: fix bug in atan(y, x) function

When x==0, the result was wrong.  Fixes piglit glsl-fs-atan-1.shader_test

ir_to_mesa: When emitting a pixel kill, flag that we did so.

Both i965 and swrast rely on UsesKill to determine whether to do early
depth writes.  Fixes glsl-fs-discard-02.

Bug #29835.

llvmpipe: slightly simplify build_mask

llvmpipe: combine linear mask calculation

llvmpipe: intrinsics versions of build_mask functions

glsl2: Write vector constructor constants in a single assignment

Make two passes over the constructor parameters.  Write all of the
constants in a single write, then write the non-constants one at a
time.  This causes the fragment shader

varying float g;
void main()
{
gl_FragColor = vec4(0.0, g, 0.0, 1.0);
}

to generate

(function main
  (signature void (parameters )
    (
      (declare (temporary ) vec4 vec_ctor@0x8580058)
      (assign (constant bool (1)) (xzw) (var_ref vec_ctor@0x8580058)  (constant vec4 (0.000000 0.000000 0.000000 1.000000)) )
      (assign (constant bool (1)) (y) (var_ref vec_ctor@0x8580058)  (swiz xxxx (var_ref g@0x8580218) ))
      (assign (constant bool (1)) (xyzw) (var_ref gl_FragColor@0x84d32a0)  (var_ref vec_ctor@0x8580058) )
    ))
)

instead of

(function main
  (signature void (parameters )
    (
      (declare (temporary ) vec4 vec_ctor@0x8580058)
      (assign (constant bool (1)) (x) (var_ref vec_ctor@0x8580058)  (constant vec4 (0.000000 0.000000 0.000000 1.000000)) )
      (assign (constant bool (1)) (y) (var_ref vec_ctor@0x8580058)  (swiz xxxx (var_ref g@0x8580218) ))
      (assign (constant bool (1)) (z) (var_ref vec_ctor@0x8580058)  (constant vec4 (0.000000 0.000000 0.000000 1.000000)) )
      (assign (constant bool (1)) (w) (var_ref vec_ctor@0x8580058)  (constant vec4 (0.000000 0.000000 0.000000 1.000000)) )
      (assign (constant bool (1)) (xyzw) (var_ref gl_FragColor@0x84d32a0)  (var_ref vec_ctor@0x8580058) )
    ))
)

A similar optimization could be done for matrix constructors, but it
is a little more complicate there.

ir_to_mesa: Sort the uniform list we're adding to Parameters[] order.

Fixes glsl-uniform-linking-1 and failure to link a shader in Unigine.
An alternative here would be to just ditch using _mesa_add_parameter
and build the initial params list on our own, but that would require
two walks of the list as well.

Bug #29822