On 7/16/25 9:20 AM, Stefan Schulze Frielinghaus wrote:
During jump bypassing also consider insns of the form

(insn 25 57 26 9 (parallel [
             (set (reg:CCZ 33 %cc)
                 (compare:CCZ (reg:SI 60 [ _9 ])
                     (const_int 0 [0])))
             (clobber (scratch:SI))
         ]) "spaceship-fp-4.c":27:1 1746 {*tstsi_cconly_extimm}
      (nil))

by testing for a single set insn during bypass_conditional_jumps().
This is a requirement for test gcc.target/s390/spaceship-fp-4.c of the
subsequent commit.

In order to silence

cprop.cc:1621:40: error: 'setcc_dest' may be used uninitialized 
[-Werror=maybe-uninitialized]
  1621 |             src = simplify_replace_rtx (src, setcc_dest, setcc_src);
       |                   ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~

initialize setcc_{dest,src} in bypass_block() although this is not
really required.

gcc/ChangeLog:

        * cprop.cc (bypass_block): Extract single set.
        (bypass_conditional_jumps): Ditto.
OK.

jeff

Reply via email to