This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Lift alignment restrictions from inlining register size memcpy


> 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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]