This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH, x86] Use vector moves in memmove expanding
- From: Michael Zolotukhin <michael dot v dot zolotukhin at gmail dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Eric Botcazou <ebotcazou at adacore dot com>, Kirill Yukhin <kirill dot yukhin at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Jan Hubicka <hubicka at ucw dot cz>, OndÅ?ej Bílka <neleai at seznam dot cz>, Uros Bizjak <ubizjak at gmail dot com>
- Date: Sat, 7 Sep 2013 00:50:14 +0400
- Subject: Re: [PATCH, x86] Use vector moves in memmove expanding
- Authentication-results: sourceware.org; auth=none
- References: <CAMe9rOrRaZANTPLZQxv8j-V5Ozy75M4d80oCC-rZRT0xQ4a=Rw at mail dot gmail dot com> <20130705112534 dot GA10164 at kam dot mff dot cuni dot cz> <CAGs3Rfs86yLj3-1HkAMnkf4ey87cZgiqajLAH=WC9mpg6OJLiw at mail dot gmail dot com> <7703907 dot JSoGtmLQkn at polaris> <CAMe9rOrphfykcfD5FApBH-Bp-Wuu8eu7qFQ+256fMgdT0WJcAA at mail dot gmail dot com>
> Michael, why did you change epilogue_size_needed to size_needed
> here? It looks wrong to me.
This function was changed in several places and meaning of
'size_needed' and 'epilogue_size_needed' could've been changed. It
needs more careful examination and I'll do it shortly.
Briefly, I significantly changed prologue and epilogue generation
routines and that obviously caused that regression. I guess that in
some of these function I began to look at size_needed instead of
desired_align (usually, size_needed >= desired_align, but for pentium
pro that's not true). I hope to dig into it on the weekend.
Thanks for reporting and investigation.