On Apr 12, 2012, Richard Guenther <[email protected]> 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
