glsl: Remove unused ir_loop_jump::loop pointer.
authorKenneth Graunke <kenneth@whitecape.org>
Sun, 1 Jul 2012 23:21:32 +0000 (16:21 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Mon, 2 Jul 2012 18:08:59 +0000 (11:08 -0700)
commitb1802a2115323233352da558430e109ff187b8af
tree4ceeb7f36a58e8f4f8230112b9270b4411d82a7c
parentd73f6375f50be6a76991e240222a8e25906bfe4b
glsl: Remove unused ir_loop_jump::loop pointer.

Commit 0c005bd7 intended to make ir_loop_jump::mode public, but also
accidentally added a new pointer to the enclosing loop.  Furthermore, it
tried to initialize the new field by adding "this->loop = loop;" to the
constructor, but since there is no loop parameter, this only initialized
the field to itself---so it will likely be a garbage pointer.

A lot of code, such as lower_jumps, allocates new loop jumps without
setting this field appropriately, so any uses would probably just crash.

Thankfully, there were none, so we can just delete the field.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=51574
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
src/glsl/ir.h