>>
>
> Please add the PR marker to the testsuite ChangeLog as well.
> I've been following this PR a bit from the sidelines, I believe a
> substantial amount of code
> (and one of the testcases) was written by Jakub, so please add him to
> the ChangeLog entries as well.
>
> This looks ok to me.
> Thanks for fixing this and thanks Jakub for the analysis and fixes too!
>
> Kyrill
>
Thanks Kyrill,
I've committed with the changelog below.
gcc/ChangeLog:
2019-02-07 Matthew Malcomson <matthew.malcom...@arm.com>
Jakub Jelinek <ja...@redhat.com>
PR bootstrap/88714
* config/arm/arm-protos.h (valid_operands_ldrd_strd,
arm_count_ldrdstrd_insns): New declarations.
* config/arm/arm.c (mem_ok_for_ldrd_strd): Remove broken handling of
MINUS.
(valid_operands_ldrd_strd): New function.
(arm_count_ldrdstrd_insns): New function.
* config/arm/ldrdstrd.md: Change peepholes to generate PARALLEL SImode
sets instead of single DImode set and define new insns to match this.
gcc/testsuite/ChangeLog:
2019-02-07 Matthew Malcomson <matthew.malcom...@arm.com>
Jakub Jelinek <ja...@redhat.com>
PR bootstrap/88714
* gcc.c-torture/execute/pr88714.c: New test.
* gcc.dg/rtl/arm/ldrd-peepholes.c: New test.
