On Sat, 21 Nov 2015, Steve Kargl wrote: > 2015-11-21 Steven G. Kargl <ka...@gcc.gnu.org> > > * simplify.c (gfc_simplify_cshift): Work around bootstrap issues > due to inappropriate warning options.
> Index: simplify.c > =================================================================== > + /* GCC bootstrap is too stupid to realize that the above code for dm > + is correct. First, dim can be specified for a rank 1 array. It is > + not needed in this nor used here. Second, the code is simply waiting > + for someone to implement rank > 1 simplification. For now, add a > + pessimization to the code that has a zero valid reason to be here. */ > + if (dm > array->rank) > + gcc_unreachable (); I'm not sure this comment is appropriate as is. At a minimum, it should list the version of GCC this was introduced for/with. So, something like "As of GCC 6, when bootstrapping we do not realize that..." Gerald
