Re: [buildrobot] [PATCH] Fix redefinition of BITS_PER_UNIT

2014-01-01 Thread Chung-Lin Tang
On 2014/1/1 02:45 PM, Mike Stump wrote:
 On Dec 31, 2013, at 12:26 PM, Jan-Benedict Glaw jbg...@lug-owl.de wrote:
 On Tue, 2013-12-31 15:24:52 +0800, Chung-Lin Tang clt...@codesourcery.com 
 wrote:
 The nios2 port was just committed. Thanks to all that gave time and
 effort to review this.

 Just a heads-up: I see a lot of warnings about BITS_PER_UNIT being
 redefined, see eg.
 http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=74923
 as an example.


 2013-12-31  Jan-Benedict Glaw  jbg...@lug-owl.de

  * config/nios2/nios2.h (BITS_PER_UNIT): Don't define it.

 diff --git a/gcc/config/nios2/nios2.h b/gcc/config/nios2/nios2.h
 index 8e6941b..f333be3 100644
 --- a/gcc/config/nios2/nios2.h
 +++ b/gcc/config/nios2/nios2.h
 @@ -73,7 +73,6 @@
 #define BITS_BIG_ENDIAN 0
 #define BYTES_BIG_ENDIAN (TARGET_BIG_ENDIAN != 0)
 #define WORDS_BIG_ENDIAN (TARGET_BIG_ENDIAN != 0)
 -#define BITS_PER_UNIT 8
 #define BITS_PER_WORD 32
 #define UNITS_PER_WORD 4
 #define POINTER_SIZE 32


 Ok?
 
 Ok.
 

Thanks for catching that.



[buildrobot] [PATCH] Fix redefinition of BITS_PER_UNIT (was: nios2 port committed)

2013-12-31 Thread Jan-Benedict Glaw
On Tue, 2013-12-31 15:24:52 +0800, Chung-Lin Tang clt...@codesourcery.com 
wrote:
 The nios2 port was just committed. Thanks to all that gave time and
 effort to review this.

Just a heads-up: I see a lot of warnings about BITS_PER_UNIT being
redefined, see eg.
http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=74923
as an example.


2013-12-31  Jan-Benedict Glaw  jbg...@lug-owl.de

* config/nios2/nios2.h (BITS_PER_UNIT): Don't define it.


diff --git a/gcc/config/nios2/nios2.h b/gcc/config/nios2/nios2.h
index 8e6941b..f333be3 100644
--- a/gcc/config/nios2/nios2.h
+++ b/gcc/config/nios2/nios2.h
@@ -73,7 +73,6 @@
 #define BITS_BIG_ENDIAN 0
 #define BYTES_BIG_ENDIAN (TARGET_BIG_ENDIAN != 0)
 #define WORDS_BIG_ENDIAN (TARGET_BIG_ENDIAN != 0)
-#define BITS_PER_UNIT 8
 #define BITS_PER_WORD 32
 #define UNITS_PER_WORD 4
 #define POINTER_SIZE 32


Ok?

MfG, JBG

-- 
  Jan-Benedict Glaw  jbg...@lug-owl.de  +49-172-7608481
Signature of: They that give up essential liberty to obtain temporary safety,
the second  : deserve neither liberty nor safety.  (Ben Franklin)


signature.asc
Description: Digital signature


Re: [buildrobot] [PATCH] Fix redefinition of BITS_PER_UNIT (was: nios2 port committed)

2013-12-31 Thread Mike Stump
On Dec 31, 2013, at 12:26 PM, Jan-Benedict Glaw jbg...@lug-owl.de wrote:
 On Tue, 2013-12-31 15:24:52 +0800, Chung-Lin Tang clt...@codesourcery.com 
 wrote:
 The nios2 port was just committed. Thanks to all that gave time and
 effort to review this.
 
 Just a heads-up: I see a lot of warnings about BITS_PER_UNIT being
 redefined, see eg.
 http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=74923
 as an example.
 
 
 2013-12-31  Jan-Benedict Glaw  jbg...@lug-owl.de
 
   * config/nios2/nios2.h (BITS_PER_UNIT): Don't define it.
 
 diff --git a/gcc/config/nios2/nios2.h b/gcc/config/nios2/nios2.h
 index 8e6941b..f333be3 100644
 --- a/gcc/config/nios2/nios2.h
 +++ b/gcc/config/nios2/nios2.h
 @@ -73,7 +73,6 @@
 #define BITS_BIG_ENDIAN 0
 #define BYTES_BIG_ENDIAN (TARGET_BIG_ENDIAN != 0)
 #define WORDS_BIG_ENDIAN (TARGET_BIG_ENDIAN != 0)
 -#define BITS_PER_UNIT 8
 #define BITS_PER_WORD 32
 #define UNITS_PER_WORD 4
 #define POINTER_SIZE 32
 
 
 Ok?

Ok.