Mesa (master): i965/cfg: Initialize bblock_t::cycle_count.

Matt Turner mattst88 at kemper.freedesktop.org
Tue Nov 29 19:58:15 UTC 2016


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

Author: Matt Turner <mattst88 at gmail.com>
Date:   Mon Nov 28 13:25:01 2016 -0800

i965/cfg: Initialize bblock_t::cycle_count.

schedule_instructions(bblock_t *) isn't called on blocks with a single
instruction, and since it is the only thing that set cycle_count,
cycle_count would be uninitialized.

A non-empty block with bblock_t::cycle_count == 0 is arguably a bug.
That'll be fixed in the next commit.

Reviewed-by: Francisco Jerez <currojerez at riseup.net>

---

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

diff --git a/src/mesa/drivers/dri/i965/brw_cfg.cpp b/src/mesa/drivers/dri/i965/brw_cfg.cpp
index ff2cf6a..fad12ee 100644
--- a/src/mesa/drivers/dri/i965/brw_cfg.cpp
+++ b/src/mesa/drivers/dri/i965/brw_cfg.cpp
@@ -51,7 +51,7 @@ link(void *mem_ctx, bblock_t *block)
 }
 
 bblock_t::bblock_t(cfg_t *cfg) :
-   cfg(cfg), idom(NULL), start_ip(0), end_ip(0), num(0)
+   cfg(cfg), idom(NULL), start_ip(0), end_ip(0), num(0), cycle_count(0)
 {
    instructions.make_empty();
    parents.make_empty();




More information about the mesa-commit mailing list