From 08f304bb3bfd4cc93bcc9108063afb7a8b2dd806 Mon Sep 17 00:00:00 2001 From: Matt Turner Date: Tue, 18 Feb 2014 16:38:07 -0800 Subject: [PATCH] i965/cfg: Allow cfg::dump to be called without a visitor. The fs_visitor's dump_instruction() implementation calls cfg_t() indirectly through calculate_live_intervals, so if you have an infinite loop in the CFG code, you can't call cfg::dump(fs_visitor *) to debug it. Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_cfg.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_cfg.cpp b/src/mesa/drivers/dri/i965/brw_cfg.cpp index 62cc23970d6..ca5b01cd03d 100644 --- a/src/mesa/drivers/dri/i965/brw_cfg.cpp +++ b/src/mesa/drivers/dri/i965/brw_cfg.cpp @@ -418,7 +418,8 @@ cfg_t::dump(backend_visitor *v) const link->block->num); } fprintf(stderr, "\n"); - block->dump(v); + if (v != NULL) + block->dump(v); fprintf(stderr, "END B%d", block->num); foreach_list_typed(bblock_link, link, link, &block->children) { fprintf(stderr, " ->B%d", -- 2.11.0