Mesa (master): i965/fs: fix exec_size when emitting DIM instruction

Samuel Iglesias Gonsálvez samuelig at kemper.freedesktop.org
Tue Jan 3 06:44:22 UTC 2017


Module: Mesa
Branch: master
Commit: c5ae6e78fc3bed83c6e18be6dbc8eb86a8db0898
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=c5ae6e78fc3bed83c6e18be6dbc8eb86a8db0898

Author: Samuel Iglesias Gonsálvez <siglesias at igalia.com>
Date:   Fri Dec 23 07:37:38 2016 +0100

i965/fs: fix exec_size when emitting DIM instruction

Otherwise, DIM instructions will be emitted with the default exec size
which could be 16 in some cases, that is not legal.

Signed-off-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>
Suggested-by: Matt Turner <mattst88 at gmail.com>
Reviewed-by: Matt Turner <mattst88 at gmail.com>

---

 src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 9f2729a..2ed843b 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
@@ -4813,7 +4813,7 @@ setup_imm_df(const fs_builder &bld, double v)
     * instruction allows to set the 64-bit immediate value.
     */
    if (devinfo->is_haswell) {
-      const fs_builder ubld = bld.exec_all();
+      const fs_builder ubld = bld.exec_all().group(1, 0);
       fs_reg dst = ubld.vgrf(BRW_REGISTER_TYPE_DF, 1);
       ubld.DIM(dst, brw_imm_df(v));
       return component(dst, 0);




More information about the mesa-commit mailing list