------- Comment #18 from matz at gcc dot gnu dot org 2010-03-23 16:46 ------- It should mostly not matter anymore as expand_expr_real_[12] and friends use an explicit location parameter, stored away before expanding operands. But to be safe, yes, expand_expr_real() should instead also restore the old location. You don't need to check for NULL gimple_block(), set_curr_insn_block does that.
-- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19192