Hello, It seems this patch causes several regressions in gfortran on ARM too: gfortran.dg/default_format_1.f90 gfortran.dg/default_format_denormal_1.f90 gfortran.dg/fmt_bz_bn.f gfortran.dg/fmt_read_bz_bn.f90 gfortran.dg/g77/f77-edit-t-in.f gfortran.dg/list_read_4.f90 gfortran.dg/namelist_11.f gfortran.dg/past_eor.f90 gfortran.dg/read_2.f90 gfortran.dg/read_float_2.f03 gfortran.dg/read_float_3.f90 gfortran.dg/read_float_4.f90 now fail after this patch.
On 8 January 2014 17:54, Uros Bizjak <ubiz...@gmail.com> wrote: > Hello! > >>> So like this instead? Bootstrapped/regtested on x86_64-linux and >>> i686-linux. For 4.8 I'd still prefer the earlier patch though. >>> >>> 2013-12-18 Jakub Jelinek <ja...@redhat.com> >>> >>> PR rtl-optimization/58668 >>> * cfgcleanup.c (flow_find_cross_jump): Don't count >>> any jumps if dir_p is NULL. Remove p1 variable, use active_insn_p >>> to determine what is counted. >>> (flow_find_head_matching_sequence): Use active_insn_p to determine >>> what is counted. >>> (try_head_merge_bb): Adjust for the flow_find_head_matching_sequence >>> counting change. >>> * ifcvt.c (count_bb_insns): Use active_insn_p && !JUMP_P to >>> determine what is counted. >>> >>> * gcc.dg/pr58668.c: New test. >> >> This is fine for the trunk. Release manager's call for what they'd prefer on >> the 4.8 branch. > > This caused PR59724 on alpha: > > 20021116-1.c: In function ‘foo’: > 20021116-1.c:31:1: error: NOTE_INSN_BASIC_BLOCK is missing for block 9 > } > ^ > 20021116-1.c:31:1: error: insn outside basic block > (jump_insn 94 52 93 9 (return) 20021116-1.c:31 -1 > (nil) > -> return) > > Uros.
