From patchwork Thu Jul 22 01:44:19 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Fix violation of self-assignment check in GCC source [4/4]
Date: Wed, 21 Jul 2010 15:44:19 -0000
From: Le-Chun Wu <lcwu@google.com>
X-Patchwork-Id: 59540
Message-Id: <AANLkTi=DQW1C=7H4k-Tec73z-328qxzSYHGurpjguLzn@mail.gmail.com>
To: GCC Patches <gcc-patches@gcc.gnu.org>, rth@redhat.com, aoliva@redhat.com

This patch fix the warning triggered by self-assignment check in
config/i386/i386.c. (Note that the compiler does not emit bogus
uninitialized warning on fcode any more so the self-init hack is no
longer needed.) Bootstrapped and tested on x86_64-gnu-linux. OK for
trunk?

Thanks,

Le-chun

2010-07-21  Le-Chun Wu  <lcwu@google.com>

       * config/i386/i386.c (ix86_vectorize_builtin_vec_perm): Remove
       unnecessary self-init.


Index: gcc/config/i386/i386.c
===================================================================
--- gcc/config/i386/i386.c      (revision 162385)
+++ gcc/config/i386/i386.c      (working copy)
@@ -30013,7 +30013,7 @@ ix86_vectorize_builtin_vec_perm (tree ve
   tree itype = TREE_TYPE (vec_type);
   bool u = TYPE_UNSIGNED (itype);
   enum machine_mode vmode = TYPE_MODE (vec_type);
-  enum ix86_builtins fcode = fcode; /* Silence bogus warning.  */
+  enum ix86_builtins fcode;
   bool ok = TARGET_SSE2;

   switch (vmode)