summaryrefslogtreecommitdiffstats
path: root/src/intel/vulkan/TODO
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/intel/vulkan/TODO
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/intel/vulkan/TODO')
0 files changed, 0 insertions, 0 deletions