[PATCH] xtensa: include elf/xtensa.h in tc-xtensa.c

Trevor Saunders tbsaunde@tbsaunde.org
Wed Jun 22 07:30:00 GMT 2016


Hi,

ping

Thanks1

Trev

On Sun, Jun 05, 2016 at 03:03:24PM -0400, tbsaunde+binutils@tbsaunde.org wrote:
> From: Trevor Saunders <tbsaunde+binutils@tbsaunde.org>
> 
> Hi,
> 
> There's no reason to define these macros twice.
> 
> built and regtested cross to xtensa-elf, ok?
> 
> Trev
> 
> gas/ChangeLog:
> 
> 2016-06-05  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>
> 
> 	* config/tc-xtensa.c: Include elf/xtensa.h.
> ---
>  gas/config/tc-xtensa.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/gas/config/tc-xtensa.c b/gas/config/tc-xtensa.c
> index 40137ff..d2a3567 100644
> --- a/gas/config/tc-xtensa.c
> +++ b/gas/config/tc-xtensa.c
> @@ -29,6 +29,7 @@
>  #include "xtensa-istack.h"
>  #include "struc-symbol.h"
>  #include "xtensa-config.h"
> +#include "elf/xtensa.h"
>  
>  /* Provide default values for new configuration settings.  */
>  #ifndef XSHAL_ABI
> @@ -213,8 +214,6 @@ int generating_literals = 0;
>  /* Required branch target alignment.  */
>  #define XTENSA_PROP_BT_ALIGN_REQUIRE    0x3
>  
> -#define GET_XTENSA_PROP_BT_ALIGN(flag) \
> -  (((unsigned) ((flag) & (XTENSA_PROP_BT_ALIGN_MASK))) >> 9)
>  #define SET_XTENSA_PROP_BT_ALIGN(flag, align) \
>    (((flag) & (~XTENSA_PROP_BT_ALIGN_MASK)) | \
>      (((align) << 9) & XTENSA_PROP_BT_ALIGN_MASK))
> @@ -235,8 +234,6 @@ int generating_literals = 0;
>  
>  #define XTENSA_PROP_ALIGNMENT_MASK      0x0001f000
>  
> -#define GET_XTENSA_PROP_ALIGNMENT(flag) \
> -  (((unsigned) ((flag) & (XTENSA_PROP_ALIGNMENT_MASK))) >> 12)
>  #define SET_XTENSA_PROP_ALIGNMENT(flag, align) \
>    (((flag) & (~XTENSA_PROP_ALIGNMENT_MASK)) | \
>      (((align) << 12) & XTENSA_PROP_ALIGNMENT_MASK))
> -- 
> 2.7.4
> 



More information about the Binutils mailing list