aboutsummaryrefslogtreecommitdiffstats
path: root/src/mesa/drivers/dri/i965/brw_clip.h
diff options
context:
space:
mode:
authorTimothy Arceri <[email protected]>2016-10-20 09:59:00 +1100
committerTimothy Arceri <[email protected]>2016-10-26 14:29:36 +1100
commit91d61fbf7cb61a44adcaae51ee08ad0dd6b2a03b (patch)
tree4fdab00ee0ee2d6e4e8a25bb3e2043e6f0b03f27 /src/mesa/drivers/dri/i965/brw_clip.h
parent20c0e67501350952191b70c907c26019e5722243 (diff)
i965: rewrite brw_setup_vue_interpolation()
Here brw_setup_vue_interpolation() is rewritten not to use the InterpQualifier array in gl_fragment_program which will allow us to remove it. This change also makes the code which is only used by gen4/5 more self contained as it now has its own gen5_fragment_program struct rather than storing the map in brw_context. This means the interpolation map will only get processed once and will get stored in the in memory cache rather than being processed everytime the fs changes. Also by calling this from the fs compile code rather than from the upload code and using the interpolation assigned there we can get rid of the BRW_NEW_INTERPOLATION_MAP flag. It might not seem ideal to add a gen5_fragment_program struct however by the end of this series we will have gotten rid of all the brw_{shader_stage}_program structs and replaced them with a generic brw_program struct so there will only be two program structs which is better than what we have now. V2: Don't remove BRW_NEW_INTERPOLATION_MAP from dirty_bit_map until the following patch to fix build error. V3 - Suggestions by Jason: - name struct gen4_fragment_program rather than gen5_fragment_program - don't use enum with memset() - create interp mode set helper and simplify logic to call it - add assert when calling function to show prog will never be NULL for gen4/5 i.e. no Vulkan Reviewed-by: Jason Ekstrand <[email protected]> Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_clip.h')
-rw-r--r--src/mesa/drivers/dri/i965/brw_clip.h7
1 files changed, 3 insertions, 4 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_clip.h b/src/mesa/drivers/dri/i965/brw_clip.h
index 54c76822e22..355ae64eefe 100644
--- a/src/mesa/drivers/dri/i965/brw_clip.h
+++ b/src/mesa/drivers/dri/i965/brw_clip.h
@@ -47,7 +47,9 @@
*/
struct brw_clip_prog_key {
GLbitfield64 attrs;
- struct interpolation_mode_map interpolation_mode;
+ bool contains_flat_varying;
+ bool contains_noperspective_varying;
+ unsigned char *interp_mode;
GLuint primitive:4;
GLuint nr_userclip:4;
GLuint pv_first:1;
@@ -128,9 +130,6 @@ struct brw_clip_compile {
bool need_direction;
struct brw_vue_map vue_map;
-
- bool has_flat_shading;
- bool has_noperspective_shading;
};
/**