tree-ssa-operands.c (add_call_clobber_ops): Fix unused variable warning with code disable.

* tree-ssa-operands.c (add_call_clobber_ops): Fix unused variable
        warning with code disable.  Fix formatting.

From-SVN: r87567
This commit is contained in:
Richard Henderson 2004-09-15 15:05:39 -07:00 committed by Richard Henderson
parent c7f9021961
commit e776368b7a
2 changed files with 36 additions and 37 deletions

View File

@ -1,3 +1,8 @@
2004-09-15 Richard Henderson <rth@redhat.com>
* tree-ssa-operands.c (add_call_clobber_ops): Fix unused variable
warning with code disable. Fix formatting.
2004-09-15 Steven Bosscher <stevenb@suse.de> 2004-09-15 Steven Bosscher <stevenb@suse.de>
* Makefile.in (tree-ssa-dom.o): Depend on tree-ssa-propagate.h. * Makefile.in (tree-ssa-dom.o): Depend on tree-ssa-propagate.h.

View File

@ -1581,58 +1581,52 @@ add_call_clobber_ops (tree stmt, tree callee)
else else
{ {
size_t i; size_t i;
bitmap not_read_b = NULL, not_written_b = NULL;
/* Get info for module level statics. There is a bit set for /* Get info for module level statics. There is a bit set for
each static if the call being processed does not read or each static if the call being processed does not read or
write that variable. */ write that variable. */
/* hack to turn off the optimization until I can get the bug fixed. */ /* ??? Turn off the optimization until it gets fixed. */
/* bitmap not_read_b = callee if (0 && callee)
? get_global_statics_not_read (callee) : NULL; {
bitmap not_written_b = callee not_read_b = get_global_statics_not_read (callee);
? get_global_statics_not_written (callee) : NULL; not_written_b = get_global_statics_not_written (callee);
*/ }
bitmap not_read_b = NULL;
bitmap not_written_b = NULL;
EXECUTE_IF_SET_IN_BITMAP (call_clobbered_vars, 0, i, EXECUTE_IF_SET_IN_BITMAP (call_clobbered_vars, 0, i,
{ {
tree var = referenced_var (i); tree var = referenced_var (i);
bool not_read = not_read_b bool not_read
? bitmap_bit_p(not_read_b, i) : false; = not_read_b ? bitmap_bit_p (not_read_b, i) : false;
bool not_written = not_written_b bool not_written
? bitmap_bit_p(not_written_b, i) : false; = not_written_b ? bitmap_bit_p (not_written_b, i) : false;
if (not_read)
if (not_read) {
{ /* The var is not read during the call. */ /* The var is not read during the call. */
if (not_written) if (!not_written)
{
/* Nothing. */
}
else
add_stmt_operand (&var, stmt, opf_is_def); add_stmt_operand (&var, stmt, opf_is_def);
} }
else else
{ /* The var is read during the call. */ {
/* The var is read during the call. */
if (not_written) if (not_written)
add_stmt_operand (&var, stmt, opf_none); add_stmt_operand (&var, stmt, opf_none);
else
/* The not_read and not_written bits are only set /* The not_read and not_written bits are only set for module
for module static variables. Neither is set static variables. Neither is set here, so we may be dealing
here, so we may be dealing with a module static with a module static or we may not. So we still must look
or we may not. So we still must look anywhere anywhere else we can (such as the TREE_READONLY) to get
else we can (such as the TREE_READONLY) to get better info. */
better info. */ /* If VAR is read-only, don't add a V_MAY_DEF, just a
/* If VAR is read-only, don't add a V_MAY_DEF, just a VUSE operand. */
VUSE operand. */ else if (TREE_READONLY (var))
if (TREE_READONLY (var)) add_stmt_operand (&var, stmt, opf_none);
add_stmt_operand (&var, stmt, opf_none); else
else add_stmt_operand (&var, stmt, opf_is_def);
add_stmt_operand (&var, stmt, opf_is_def); }
}
}); });
} }
} }