diff mbox

Fix building microblaze targets with trunk

Message ID 560AC3E5.7090603@redhat.com
State New
Headers show

Commit Message

Jeff Law Sept. 29, 2015, 5:01 p.m. UTC
The microblaze port as a "*p++" statement which computes a result that 
is never used (the memory result).  This removes the spurious memory 
dereference and the unused value warning.

Tested by building the microblaze targets in config-all.mk.

Installed on the trunk.

Jeff
commit b2e58a1a53a3bbba60bd39ce53beb9fd706742f4
Author: Jeff Law <law@tor.usersys.redhat.com>
Date:   Tue Sep 29 11:59:11 2015 -0400

    [PATCH] Fix building microblaze targets with trunk
    	* config/microblaze/microblaze.c (microblaze_version_to_int): Remove
    	computation of unused value.

Comments

Michael Eager Sept. 29, 2015, 5:08 p.m. UTC | #1
On 09/29/2015 10:01 AM, Jeff Law wrote:
> The microblaze port as a "*p++" statement which computes a result that is never used (the memory
> result).  This removes the spurious memory dereference and the unused value warning.
>
> Tested by building the microblaze targets in config-all.mk.
>
> Installed on the trunk.


OK.
diff mbox

Patch

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 13e930a..8d55423 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,8 @@ 
 2015-09-29  Jeff Law  <law@redhat.com>
 
+	* config/microblaze/microblaze.c (microblaze_version_to_int): Remove
+	computation of unused value.
+
 	* config/pdp11/pdp11.c (pdp11_branch_cost): New function.
 	* config/pdp11/pdp11.h (BRANCH_COST): Call function rather than
 	inline macro expansion.
diff --git a/gcc/config/microblaze/microblaze.c b/gcc/config/microblaze/microblaze.c
index 6e7745a..ebcf65a 100644
--- a/gcc/config/microblaze/microblaze.c
+++ b/gcc/config/microblaze/microblaze.c
@@ -1640,7 +1640,7 @@  microblaze_version_to_int (const char *version)
 	{			/* Looking for major  */
           if (*p == '.')
             {
-              *v++;
+              v++;
             }
           else
             {