This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, arm, testsuite] fix regression in test di-longlong64-sync-withldrexd.c
- From: Mike Stump <mikestump at comcast dot net>
- To: Greta Yorsh <Greta dot Yorsh at arm dot com>
- Cc: <ramana at gcc dot gnu dot org>, <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 25 Jan 2012 10:14:15 -0800
- Subject: Re: [patch, arm, testsuite] fix regression in test di-longlong64-sync-withldrexd.c
- References: <001901ccdb76$fd38be40$f7aa3ac0$@Yorsh@arm.com>
On Jan 25, 2012, at 7:35 AM, Greta Yorsh wrote:
> The test gcc.target/arm/di-longlong64-sync-withldrexd.c fails on
> arm-none-eabi target, because gcc generates 48 LDREXD and 48 STREXD
> instructions instead of the expected 46.
>
> FAIL: gcc.target/arm/di-longlong64-sync-withldrexd.c scan-assembler-times
> tldrexd 46
> FAIL: gcc.target/arm/di-longlong64-sync-withldrexd.c scan-assembler-times
> tstrexd 46
> The regression PASS->FAIL was introduced for target arm-none-eabi in the
> first week of November 2011.
Gosh, that seems like a long time to notice and adjust the testcase for a port that seems popular enough and has a nice representation around here. :-(
> * gcc.target/arm/di-longlong64-sync-withldrexd.c:
Please say slightly more than this.
Also, if you mean to ask for a review, please include an Ok? in there.
I'll assume you meant to ask for a review. Ok. If you aren't absolutely sure about the codegen or have an doubts, please ping an arm person for review of the codegen.