[patch, arm, testsuite] fix regression in test di-longlong64-sync-withldrexd.c

Mike Stump mikestump@comcast.net
Wed Jan 25 18:14:00 GMT 2012


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.



More information about the Gcc-patches mailing list