[Bug middle-end/21237] [4.0/4.1 Regression] unnecessary __cmpdi2 usage?

[Bug middle-end/21237] [4.0/4.1 Regression] unnecessary __cmpdi2 usage?

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-05-09 14:46 -------
Subject: Bug 21237

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	dje@gcc.gnu.org	2005-05-09 14:45:59

Modified files:
	gcc            : ChangeLog 

Log message:
	PR middle-end/21237
	* dojump.c (do_jump, MINUS_EXPR): Build NE_EXPR and fall through
	to NE_EXPR case.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.8673&r2=2.8674



-- 


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

[Bug middle-end/21237] [4.0 Regression] unnecessary __cmpdi2 usage?

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-05-14 00:46 -------
Subject: Bug 21237

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-4_0-branch
Changes by:	dje@gcc.gnu.org	2005-05-14 00:46:01

Modified files:
	gcc            : ChangeLog dojump.c 

Log message:
	PR middle-end/21237
	* dojump.c (do_jump, MINUS_EXPR): Build NE_EXPR and fall through
	to NE_EXPR case.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=2.7592.2.244&r2=2.7592.2.245
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/dojump.c.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.37&r2=1.37.8.1



-- 


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

[Bug middle-end/21237] [4.0 Regression] unnecessary __cmpdi2 usage?

[dje at gcc dot gnu dot org]

------- Additional Comments From dje at gcc dot gnu dot org  2005-05-14 00:48 -------
backported patch

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


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

[Bug middle-end/21237] [4.0/4.1 Regression] unnecessary __cmpdi2 usage?

[dje at gcc dot gnu dot org]

------- Additional Comments From dje at gcc dot gnu dot org  2005-05-07 20:24 -------
i386 synthesizes a DImode compare for 32-bit mode in target-dependent code. 
PowerPC probably needs to do the same.

-- 


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

[Bug middle-end/21237] [4.0/4.1 Regression] unnecessary __cmpdi2 usage?

[pluto at pld-linux dot org]

------- Additional Comments From pluto at pld-linux dot org  2005-04-28 18:04 -------
__cmdi2 isn't used with -O0 but it is with -O1. 

-- 


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

[Bug middle-end/21237] [4.0/4.1 Regression] unnecessary __cmpdi2 usage?

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-04-26 23:07 -------
Confirmed, reduced testcase:
int g(long long a, long long b)
{
  a -= b;
  return !a;
}

You were looking at the wrong function as we inline a function into cryptcompress_truncate which 
causes this.

One issue is when we go and expand (a - b) == 0, we expand it to cmpdi for some reason instead of 
inlining cmpdi.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
  GCC build triplet|i686-pld-linux              |
   GCC host triplet|i686-pld-linux              |
 GCC target triplet|ppc-pld-linux               |powerpc-*-*
   Last reconfirmed|0000-00-00 00:00:00         |2005-04-26 23:07:07
               date|                            |
            Summary|unnecessary __cmpdi2 usage? |[4.0/4.1 Regression]
                   |                            |unnecessary __cmpdi2 usage?
   Target Milestone|---                         |4.0.1


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