- Sep 14, 2010
-
-
Andre Maasikas authored
when setting negative integers to bitfields we could overwrite other parts of it. So mask the value to be written correctly. This is used quite often in the driver - hope it doesnt affect performace or uncover behaviour relied before... fixes strange effects when setting negative lodbias on evergreen
-
Brian Paul authored
-
Brian Paul authored
-
Brian Paul authored
A number of other files had to be updated as well because const qualifiers were added to the glMultiDrawArrays() function. Also, GL_FIXED is now defined in glext.h.
-
Brian Paul authored
-
Brian Paul authored
Silences a compiler warning. Still need to add some assertions for this case.
-
Brian Paul authored
-
Chia-I Wu authored
Remove all FEATURE tests in mesa/drivers/common/. They are not needed and the code looks better without them.
-
Chia-I Wu authored
Add dummy static inline definitions to syncobj.h when FEATURE_ARB_sync is 0, and remove most FEATURE_ARB_sync tests.
-
Chia-I Wu authored
They were intended to be used to build OpenGL ES only DRI drivers, but that never happened.
-
Vinson Lee authored
-
Vinson Lee authored
-
Vinson Lee authored
Fixes the following GCC warning. brw_screen.c: In function 'brw_get_shader_param': brw_screen.c:241: warning: control reaches end of non-void function
-
Vinson Lee authored
Fixes the following GCC warning. i915_screen.c: In function 'i915_get_shader_param': i915_screen.c:184: warning: control reaches end of non-void function
-
Vinson Lee authored
Fixes the following GCC warning. i915_screen.c: In function 'i915_get_shader_param': i915_screen.c:147: warning: implicit declaration of function 'draw_get_shader_param'
-
Vinson Lee authored
This is a follow-up to commit a508d2dd. Fixes the following GCC warning. id_screen.c: In function 'identity_screen_create': id_screen.c:317: warning: assignment from incompatible pointer type
-
Vinson Lee authored
This is a follow-up to commit a508d2dd. Fixes the following GCC warning. rbug_screen.c: In function 'rbug_screen_create': rbug_screen.c:331: warning: assignment from incompatible pointer type
-
Luca Barbieri authored
Note, BTW, that the Gallium implementation returns 0.5, which seems to violate the GLSL spec, where it should return 0.0 instead. Not sure whether changing it to 0 is correct or not.
-
Luca Barbieri authored
This turns on if conversion and unlimited loop unrolling if control flow is not supported. NOTE: this will change the behavior of r300g and any other driver that doesn't advertise control flow
-
Luca Barbieri authored
Changes in v3: - Also change trace, which I forgot about Changes in v2: - No longer adds tessellation shaders Currently each shader cap has FS and VS versions. However, we want a version of them for geometry, tessellation control, and tessellation evaluation shaders, and want to be able to easily query a given cap type for a given shader stage. Since having 5 duplicates of each shader cap is unmanageable, add a new get_shader_param function that takes both a shader cap from a new enum and a shader stage. Drivers with non-unified shaders will first switch on the shader and, within each case, switch on the cap. Drivers with unified shaders instead first check whether the shader is supported, and then switch on the cap. MAX_CONST_BUFFERS is now per-stage. The geometry shader cap is removed in favor of checking whether the limit of geometry shader instructions is greater than 0, which is also used for tessellation shaders. WARNING: all drivers changed and compiled but only nvfx tested
-
Ian Romanick authored
-
Luca Barbieri authored
Currently GLSL IR forbids any vector comparisons, and defines "ir_binop_equal" and "ir_binop_nequal" to compare all elements and give a single bool. This is highly unintuitive and prevents generation of optimal Mesa IR. Hence, first rename "ir_binop_equal" to "ir_binop_all_equal" and "ir_binop_nequal" to "ir_binop_any_nequal". Second, readd "ir_binop_equal" and "ir_binop_nequal" with the same semantics as less, lequal, etc. Third, allow all comparisons to acts on vectors. Signed-off-by:
Ian Romanick <ian.d.romanick@intel.com>
-
- Sep 13, 2010
-
-
Luca Barbieri authored
If the loop ends with an if with one break or in a single break unroll it. Loops that end with a continue will have that continue removed by the redundant jump optimizer. Likewise loops that end with an if-statement with a break at the end of both branches will have the break pulled out after the if-statement. Loops of the form for (...) { do_something1(); if (cond) { do_something2(); break; } else { do_something3(); } } will be unrolled as do_something1(); if (cond) { do_something2(); } else { do_something3(); do_something1(); if (cond) { do_something2(); } else { do_something3(); /* Repeat inserting iterations here.*/ } } ir_lower_jumps can guarantee that all loops are put in this form and thus all loops are now potentially unrollable if an upper bound on the number of iterations can be found. Signed-off-by:
Ian Romanick <ian.d.romanick@intel.com>
-
Ian Romanick authored
-
Ian Romanick authored
-
Luca Barbieri authored
The loop_controls pass didn't look at the counter values it put in ir_loop on previous iterations, so while the first iteration worked, subsequent ones couldn't determine max_iterations.
-
Ian Romanick authored
Fixes piglit tests glsl-vs-main-return and glsl-fs-main-return.
-
Luca Barbieri authored
-
Luca Barbieri authored
Changes in v2: - Base class renamed to ir_control_flow_visitor - Tried to comply with coding style This is a new pass that supersedes ir_if_return and "lowers" jumps to if/else structures. Currently it causes no regressions on softpipe and nv40, but I'm not sure whether the piglit glsl tests are thorough enough, so consider this experimental. It can be asked to: 1. Pull jumps out of ifs where possible 2. Remove all "continue"s, replacing them with an "execute flag" 3. Replace all "break" with a single conditional one at the end of the loop 4. Replace all "return"s with a single return at the end of the function, for the main function and/or other functions This gives several great benefits: 1. All functions can be inlined after this pass 2. nv40 and other pre-DX10 chips without "continue" can be supported 3. nv30 and other pre-DX10 chips with no control flow at all are better supported Note that for full effect we should also teach the unroller to unroll loops with a fixed maximum number of iterations but with the canonical conditional "break" that this pass will insert if asked to. Continues are lowered by adding a per-loop "execute flag", initialized to TRUE, that when cleared inhibits all execution until the end of the loop. Breaks are lowered to continues, plus setting a "break flag" that is checked at the end of the loop, and trigger the unique "break". Returns are lowered to breaks/continues, plus adding a "return flag" that causes loops to break again out of their enclosing loops until all the loops are exited: then the "execute flag" logic will ignore everything until the end of the function. Note that "continue" and "return" can also be implemented by adding a dummy loop and using break. However, this is bad for hardware with limited nesting depth, and prevents further optimization, and thus is not currently performed.
-
Luca Barbieri authored
This is just a subclass of ir_visitor with empty implementations of all the visit methods for non-control flow nodes. Used to avoid duplicating that in ir_visitor subclasses. ir_hierarchical_visitor is another way to solve this, but is less natural for some applications.
-
Jose Fonseca authored
Should fix fdo 30168.
-
According to gcc documentation both are equivalent, second are prefered as first can make conflict with existing symbols. Signed-off-by:
José Fonseca <jfonseca@vmware.com>
-
Jesse Barnes authored
Point about needing a better way to do this validated.
-
Alex Deucher authored
This applies to r6xx/r7xx/evergreen
-
Jesse Barnes authored
Allows KMS EGL driver to load. We need a better way of doing this.
-
Alex Deucher authored
-
Alex Deucher authored
-
Alex Deucher authored
saves a few dwords
-
Alex Deucher authored
r700start3d already emits the context control packets
-