Skip to content

intel/brw: Fix limiting Xe2 to SIMD16 when dual source blend is enabled

Jordan Justen requested to merge jljusten/mesa:xe2-simd16-dual-source-limit into main

@zachary.battleman was looking at @tpalli's comment added in 207eb944, trying to determine if more places should use intel_needs_workaround(). He noticed that @currojerez's code path that I merged in af8b9af7 wouldn't ever be taken.

This patch should fix that, and I moved Tapani's comment inside the if block which might be a little clearer.

intel/brw: Fix limiting Xe2 to SIMD16 when dual source blend is enabled

I also moved Tapani's comment from 207eb944450 inside the if block so
it doesn't appear like the only purpose of the block is to implement
the referenced workaround.

Found-by: Zach Battleman
Ref: 207eb944450 ("intel/compiler: add comment about workaround on simd width")
Fixes: af8b9af700e ("intel/brw/xe2+: Allow dual-source blending in SIMD16 mode.")
Signed-off-by: Jordan Justen

Merge request reports