------- Comment #3 from ramana dot radhakrishnan at celunite dot com 2007-07-10 20:14 ------- (In reply to comment #2) > As the decl is volatile as well this is clearly a bogus optimization. >
Putting a breakpoint on evaluate_stmt in tree-ssa-ccp.c shows that stmt_ann of the stmt does not have has_volatile_ops set to true. (gdb) p stmt (gdb) pt <gimple_modify_stmt 0xb7d4fee0 side-effects arg 0 <ssa_name 0xb7d4aed4 type <pointer_type 0xb7d44bd0 type <integer_type 0xb7d44b64 int> sizes-gimplified unsigned SI size <integer_cst 0xb7caa460 constant invariant 32> unit size <integer_cst 0xb7caa24c constant invariant 4> align 32 symtab 0 alias set -1 canonical type 0xb7d44bd0> visited var <var_decl 0xb7d5005c spinlock0> def_stmt <gimple_modify_stmt 0xb7d4fee0> version 1> arg 1 <addr_expr 0xb7d481c0 type <pointer_type 0xb7d44bd0> constant invariant arg 0 <array_ref 0xb7cb5084 type <integer_type 0xb7d44b64 int> arg 0 <var_decl 0xb7d50000 spinlock> arg 1 <integer_cst 0xb7d4fec4 constant invariant 0>>> try.c:5> (gdb) (gdb) p *(stmt->base->ann) $17 = {common = {type = STMT_ANN, aux = 0x0, value_handle = 0x0}, vdecl = { common = {type = STMT_ANN, aux = 0x0, value_handle = 0x0}, out_of_ssa_tag = 0, base_var_processed = 0, used = 0, need_phi_state = NEED_PHI_STATE_NO, in_vuse_list = 1, in_vdef_list = 0, is_heapvar = 0, call_clobbered = 1, noalias_state = NO_ALIAS_GLOBAL, mpt = 0xb7cb6804, symbol_mem_tag = 0x0, partition = 0, base_index = 0, current_def = 0x0, subvars = 0x0, escape_mask = 3084210192}, fdecl = { common = {type = STMT_ANN, aux = 0x0, value_handle = 0x0}, reference_vars_info = 0xb7eed528}, stmt = {common = {type = STMT_ANN, aux = 0x0, value_handle = 0x0}, bb = 0xb7eed528, operands = { def_ops = 0xb7cb6804, use_ops = 0x0, vdef_ops = 0x0, vuse_ops = 0x0, stores = 0x0, loads = 0x0}, addresses_taken = 0xb7d55010, uid = 0, references_memory = 0, modified = 0, has_volatile_ops = 0, makes_clobbering_call = 0}} Shouldn't has_volatile_ops get set to true in this case because the stmt essentially has one volatile operand here ? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32721