http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58493

Mikael Pettersson <mikpe at it dot uu.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mikpe at it dot uu.se

--- Comment #3 from Mikael Pettersson <mikpe at it dot uu.se> ---
The C test case started working for 4.9 with Richard's "Vectorizer TLC:
re-organize data dependence checking" patch in r196872.  The original patch
description <http://gcc.gnu.org/ml/gcc-patches/2013-02/msg01238.html> doesn't
mention any wrong-code fixes in that patch.

The code generation difference on the test case at r196872 is:

--- pr54893.s-r196871   2013-09-23 14:51:29.441292880 +0200
+++ pr54893.s-r196872   2013-09-23 14:54:55.000501714 +0200
@@ -12,8 +12,6 @@
        sarl    %edi
        testl   %edi, %edi
        jle     .L25
-       cmpl    $8, %edi
-       jbe     .L3
        leaq    64(%rsi), %rax
        cmpq    %rax, %rdx
        leaq    64(%rdx), %rax
@@ -22,6 +20,8 @@
        setae   %al
        orb     %al, %cl
        je      .L3
+       cmpl    $9, %edi
+       jbe     .L3
        leal    -1(%rdi), %r9d
        vmovapd .LC0(%rip), %ymm1
        movq    %rdx, %rax

Reply via email to