This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH][Testsuite] Use user defined memmove in gcc.c-torture/execute/builtins/memops-asm-lib.c
- From: Martin Sebor <msebor at gmail dot com>
- To: Renlin Li <renlin dot li at foss dot arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, richard dot sandiford at arm dot com
- Date: Fri, 23 Jun 2017 09:27:08 -0600
- Subject: Re: [PATCH][Testsuite] Use user defined memmove in gcc.c-torture/execute/builtins/memops-asm-lib.c
- Authentication-results: sourceware.org; auth=none
- References: <594CDD3A.email@example.com>
On 06/23/2017 03:19 AM, Renlin Li wrote:
After the change r249278. bcopy is folded into memmove. And in newlib
memmove implementation, it will call memcpy in certain conditions.
The memcpy defined in memops-asm-lib.c will abort when the test is running.
In this case, I defined a user memmove function which by pass the
So that memcpy won't be called accidentally.
Okay to commit?
Having memmove call memcpy when there is no overlap seems like
a valid transformation. I don't know which test specifically
fails so the question on my mind is whether it perhaps is overly
restrictive in assuming that this transformation must never take
place. Other than that, although I can't really approve patches,
this one looks okay to me. Thanks for getting to the bottom of
the failure and fixing it!
2017-06-22 Renlin Li <firstname.lastname@example.org>
Szabolcs Nagy <email@example.com>
* gcc.c-torture/execute/builtins/memops-asm-lib.c (my_memmove): New.
* gcc.c-torture/execute/builtins/memops-asm.c (memmove): Declare