This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Lift alignment restrictions from inlining register size memcpy
- From: Eric Botcazou <ebotcazou at adacore dot com>
- To: Richard Biener <rguenther at suse dot de>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 12 Jul 2016 23:37:17 +0200
- Subject: Re: [PATCH] Lift alignment restrictions from inlining register size memcpy
- Authentication-results: sourceware.org; auth=none
- References: <alpine.LSU.2.11.1607121131260.30444@t29.fhfr.qr>
> I have added a testcase that should exercise most relevant cases so
> we can look for fallout on "interesting" targets.
It passes on SPARC/Solaris 32-bit.
> Boostrap / regtest on strict-alignment platforms welcome.
Preliminary testing looks good, with a few expected regressions I guess:
+XPASS: gcc.dg/memmove-4.c scan-tree-dump-not optimized "memmove"
+FAIL: gcc.dg/strlenopt-8.c scan-tree-dump-times strlen "strlen \\\\(" 0
+FAIL: gcc.dg/strlenopt-8.c scan-tree-dump-times strlen "memcpy \\\\(" 4
> Unless I hear some positives I'll let the patch sit here as I
> don't really care too much about those pesky targets (and
> targets can choose to opt-in by providing movmisalign optabs
> anyway so they don't go the store/extract_bit_field path).
I'll conduct more thorough testing over the next few days.
--
Eric Botcazou