Kenneth Graunke <kenn...@whitecape.org> writes: > 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 > Cc: Ian Romanick <ian.d.roman...@intel.com> > Cc: Eric Anholt <e...@anholt.net> > Cc: tha...@chromium.org
Reviewed-by: Eric Anholt <e...@anholt.net>
pgpreAxastXOs.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
