[Mesa-dev] [PATCH 11/19] i965/fs: Move setting opcode = NOP to its one useful location.

Matt Turner mattst88 at gmail.com
Thu Feb 20 13:41:24 PST 2014


All other callers of init() immediately set opcode to something else.
---
 src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp
index e51cdbd..37b5bb0 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
@@ -55,7 +55,6 @@ void
 fs_inst::init()
 {
    memset(this, 0, sizeof(*this));
-   this->opcode = BRW_OPCODE_NOP;
 
    /* This will be the case for almost all instructions. */
    this->regs_written = 1;
@@ -64,6 +63,7 @@ fs_inst::init()
 fs_inst::fs_inst()
 {
    init();
+   this->opcode = BRW_OPCODE_NOP;
 }
 
 fs_inst::fs_inst(enum opcode opcode)
-- 
1.8.3.2



More information about the mesa-dev mailing list