On Tue, Oct 6, 2015 at 10:01 AM, Kenneth Graunke <kenn...@whitecape.org> wrote:
> Switch statements introduce a bogus loop with an unconditional break at
> the end of the loop, just before the while...so the while is unreachable
> and has no immediate dominator.
>
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> ---
> src/mesa/drivers/dri/i965/brw_cfg.cpp | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_cfg.cpp
> b/src/mesa/drivers/dri/i965/brw_cfg.cpp
> index 91d53ef..97e0624 100644
> --- a/src/mesa/drivers/dri/i965/brw_cfg.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_cfg.cpp
> @@ -422,7 +422,11 @@ cfg_t::dump(backend_shader *s)
> calculate_idom();
>
> foreach_block (block, this) {
> - fprintf(stderr, "START B%d IDOM(B%d)", block->num, block->idom->num);
> + if (block->idom)
> + fprintf(stderr, "START B%d IDOM(B%d)", block->num,
> block->idom->num);
> + else
> + fprintf(stderr, "START B%d IDOM(NONE?!)", block->num);
s/NONE?!/none/ and
Reviewed-by: Matt Turner <matts...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
