On Tue, Oct 24, 2017 at 5:39 PM, Rainer Orth
<r...@cebitec.uni-bielefeld.de> wrote:
> Hi Uros,
>
>> Remove dead code, obsoleted by cbranch rewrite years ago.
>>
>> 2017-10-11 Uros Bizjak <ubiz...@gmail.com>
>>
>> * config/i386/i386.md (*cmp<X87MODEF:mode>_<SWI24:mode>_i387):
>> Do not use float_operator operator predicate.
>> (*cmp<X87MODEF:mode>_<SWI24:mode>_cc_i387): Ditto.
>> * config/i386/predicates.md (float_operator): Remove predicate.
>>
>> 2017-10-11 Uros Bizjak <ubiz...@gmail.com>
>>
>> * config/i386/i386.md (*jcc<mode>_0_i387): Remove insn pattern.
>> (*jccxf_i387): Ditto.
>> (*jcc<mode>_i387): Ditto.
>> (*jccu<mode>_i387): Ditto.
>> (*jcc<X87MODEF:mode>_<SWI24:mode>_i387): Ditto.
>> (*jcc_*_i387 splitters): Remove.
>> * config/i386/i386-protos.h (ix86_split_fp_branch): Remove prototype.
>> * config/i386/i386.c (ix86_split_fp_branch): Remove.
>> * config/i386/predicates.md (ix86_swapped_fp_comparison_operator):
>> Remove predicate.
>>
>> testsuite/ChangeLog:
>>
>> 2017-10-11 Uros Bizjak <ubiz...@gmail.com>
>>
>> * gcc.target/i386/387-ficom-1.c: New test.
>>
>> Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.
>>
>> Committed to mainline SVN.
>
> the new testcase and its 387-ficom-2.c companion FAIL on Solaris/x86
> with /bin/as:
>
> +FAIL: gcc.target/i386/387-ficom-1.c scan-assembler-times ficomps 3 (found 0
> times)
> +FAIL: gcc.target/i386/387-ficom-2.c scan-assembler-times ficomps 3 (found 0
> times)
>
> With gas, it's
>
> ficomps s
> ficomps s
> ficomps 4(%esp)
>
> while for Solaris as this is used:
>
> ficomp s
> ficomp s
> ficomp 4(%esp)
>
> The following patch accounts for the difference. Tested with the
> appropriate runtest invocation on i386-pc-solaris2.11 and
> x86_64-pc-linux-gnu. Ok for mainline?
>
> Rainer
>
> --
> -----------------------------------------------------------------------------
> Rainer Orth, Center for Biotechnology, Bielefeld University
>
>
> 2017-10-24 Rainer Orth <r...@cebitec.uni-bielefeld.de>
>
> * gcc.target/i386/387-ficom-1.c: Allow for ficomp without s
> suffix.
> * gcc.target/i386/387-ficom-2.c: Likewise.
OK.
Thanks,
Uros.
