[Bug bootstrap/38288] New: i386/i386.c: 7 * set but not used variables

[Bug bootstrap/38288] New: i386/i386.c: 7 * set but not used variables

[dcb314 at hotmail dot com]

I just tried to build the GNU gcc version 4.4.0 snapshot 20081121
with the Intel C compiler, which said

../../src/gcc-4.4-20081121/gcc/config/i386/i386.c(4315): warning #593: variable
"f" was set but never used
../../src/gcc-4.4-20081121/gcc/config/i386/i386.c(7479): warning #593: variable
"total_size" was set but never used
../../src/gcc-4.4-20081121/gcc/config/i386/i386.c(8886): warning #593: variable
"reason" was set but never used
../../src/gcc-4.4-20081121/gcc/config/i386/i386.c(8887): warning #593: variable
"reason_rtx" was set but never used
../../src/gcc-4.4-20081121/gcc/config/i386/i386.c(13074): warning #593:
variable "op1" was set but never used
../../src/gcc-4.4-20081121/gcc/config/i386/i386.c(13520): warning #593:
variable "arithmetics_cost"was set but never used
../../src/gcc-4.4-20081121/gcc/config/i386/i386.c(16432): warning #593:
variable "piece_size_mask" was set but never used

I checked the source code and I agree with the Intel compiler.
Suggest delete all seven set but not used variables.


-- 
           Summary: i386/i386.c: 7 * set but not used variables
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Severity: minor
          Priority: P3
         Component: bootstrap
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: dcb314 at hotmail dot com
  GCC host triplet: suse-linux-x86_64


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

[Bug target/38288] i386/i386.c: 7 * set but not used variables

[dcb314 at hotmail dot com]

------- Comment #1 from dcb314 at hotmail dot com  2008-11-28 10:59 -------

I've sent a patch to gcc-patches.

If the patch is accepted, then this report may
be closed.


-- 


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

[Bug target/38288] i386/i386.c: 7 * set but not used variables

[ubizjak at gmail dot com]

------- Comment #2 from ubizjak at gmail dot com  2008-11-28 13:28 -------
(In reply to comment #1)
> I've sent a patch to gcc-patches.

Can you paste the URL to the patch?


-- 


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

[Bug target/38288] i386/i386.c: 7 * set but not used variables

[dcb314 at hotmail dot com]

------- Comment #3 from dcb314 at hotmail dot com  2008-11-28 13:55 -------
Created an attachment (id=16786)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16786&action=view)
patch for bug report


-- 


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

[Bug target/38288] i386/i386.c: 7 * set but not used variables

[ubizjak at gmail dot com]

------- Comment #4 from ubizjak at gmail dot com  2009-09-17 09:14 -------
Is the patch from Comment #3 committed to SVN?


-- 


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

[Bug target/38288] i386/i386.c: 7 * set but not used variables

[dcb314 at hotmail dot com]

------- Comment #5 from dcb314 at hotmail dot com  2009-09-17 20:35 -------
(In reply to comment #4)
> Is the patch from Comment #3 committed to SVN?

I don't think so. I wouldn't know how to do that,
and I don't think anyone has done it for me.


-- 


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

[Bug target/38288] i386/i386.c: 7 * set but not used variables

[uros at gcc dot gnu dot org]

------- Comment #6 from uros at gcc dot gnu dot org  2009-09-18 15:13 -------
Subject: Bug 38288

Author: uros
Date: Fri Sep 18 15:12:48 2009
New Revision: 151854

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151854
Log:
        PR target/38288
        From David Binderman <dcb314@hotmail.com>:
        * gcc/config/i386/i386.c (ix86_function_regparm): Remove useless
        local variable f.  Remove stale comments.
        (ix86_compute_frame_layout): Remove useless local variable total_size.
        Remove #if 0'd code.
        (legitimate_address_p): Remove useless local variables
        reason_rtx and reason.
        (ix86_split_copysign_const): Remove useless local variable op1.
        (scale_counter): Remove useless local variable piece_size_mask.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/i386/i386.c


-- 


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

[Bug target/38288] i386/i386.c: 7 * set but not used variables

[ubizjak at gmail dot com]

------- Comment #7 from ubizjak at gmail dot com  2009-09-18 15:20 -------
Fixed.


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.5.0


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