My recent patch to use QImode for bit offsets in ZERO_EXTRACTs and

SIGN_EXTRACTs in the i386 backend shouldn't have resulted in any change

behaviour, but as reported by Rainer it produces a bootstrap failure in

gm2.  This reverts the problematic patch whilst we investigate the

underlying cause.

 

Committed as obvious.

 

 

2023-07-23  Roger Sayle  <roger@nextmovesoftware.com>

 

gcc/ChangeLog

        PR target/110787

        PR target/110790

        Revert patch.

        * config/i386/i386.md (extv<mode>): Use QImode for offsets.

        (extzv<mode>): Likewise.

        (insv<mode>): Likewise.

        (*testqi_ext_3): Likewise.

        (*btr<mode>_2): Likewise.

        (define_split): Likewise.

        (*btsq_imm): Likewise.

        (*btrq_imm): Likewise.

        (*btcq_imm): Likewise.

        (define_peephole2 x3): Likewise.

        (*bt<mode>): Likewise

        (*bt<mode>_mask): New define_insn_and_split.

        (*jcc_bt<mode>): Use QImode for offsets.

        (*jcc_bt<mode>_1): Delete obsolete pattern.

        (*jcc_bt<mode>_mask): Use QImode offsets.

        (*jcc_bt<mode>_mask_1): Likewise.

        (define_split): Likewise.

        (*bt<mode>_setcqi): Likewise.

        (*bt<mode>_setncqi): Likewise.

        (*bt<mode>_setnc<mode>): Likewise.

        (*bt<mode>_setncqi_2): Likewise.

        (*bt<mode>_setc<mode>_mask): New define_insn_and_split.

        (bmi2_bzhi_<mode>3): Use QImode offsets.

        (*bmi2_bzhi_<mode>3): Likewise.

        (*bmi2_bzhi_<mode>3_1): Likewise.

        (*bmi2_bzhi_<mode>3_1_ccz): Likewise.

        (@tbm_bextri_<mode>): Likewise.

 

 

Sorry for the inconvenience,

Roger

--