On Apr 12, 2012, Richard Guenther <richard.guent...@gmail.com> wrote:
> + /* If the conditions in which this function uses VALUE change, > + adjust gimple_replace_lhs_wants_value(). */ > + gcc_assert (gimple_replace_lhs_wants_value () > + == MAY_HAVE_DEBUG_STMTS); > + if (MAY_HAVE_DEBUG_STMTS) { > that looks ... odd. Indeed, it does. Does this look any better? bool save_value = MAY_HAVE_DEBUG_STMTS; /* If the condition above, in which this function uses VALUE change, adjust gimple_replace_lhs_wants_value() to match. The assert below helps enforce this. */ gcc_checking_assert (gimple_replace_lhs_wants_value () == save_value); if (save_value) { -- Alexandre Oliva, freedom fighter http://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin America board member Free Software Evangelist Red Hat Brazil Compiler Engineer