http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60429
--- Comment #30 from Richard Biener <rguenth at gcc dot gnu.org> --- Author: rguenth Date: Mon Mar 24 08:42:37 2014 New Revision: 208784 URL: http://gcc.gnu.org/viewcvs?rev=208784&root=gcc&view=rev Log: 2014-03-24 Richard Biener <rguent...@suse.de> PR tree-optimization/60429 * tree-ssa-structalias.c (get_constraint_for_ptr_offset): Remove duplicated line. Modified: branches/gcc-4_8-branch/gcc/ChangeLog branches/gcc-4_8-branch/gcc/tree-ssa-structalias.c --- Comment #31 from rguenther at suse dot de <rguenther at suse dot de> --- On Sat, 22 Mar 2014, mikpelinux at gmail dot com wrote: > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60429 > > Mikael Pettersson <mikpelinux at gmail dot com> changed: > > What |Removed |Added > ---------------------------------------------------------------------------- > CC| |mikpelinux at gmail dot com > > --- Comment #29 from Mikael Pettersson <mikpelinux at gmail dot com> --- > Richard, your 4.8 backport performs the same call twice in a row on lines 3017 > and 3018 in tree-ssa-structalias.c; is that really intentional? > > + /* We have to include all fields that overlap the current > + field shifted by rhsoffset. And we include at least > + the last or the first field of the variable to represent > + reachability of off-bound addresses, in particular &object + 1, > + conservatively correct. */ > + temp = first_or_preceding_vi_for_offset (curr, offset); > + temp = first_or_preceding_vi_for_offset (curr, offset); No, fixed.
