This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Ada] Consolidate target pairs for SPARC


Eric Botcazou <ebotcazou@adacore.com> writes:

>    ifeq ($(strip $(filter-out sparc sun solaris%,$(targ))),)
>      ifeq ($(strip $(MULTISUBDIR)),/sparcv9)
> -      LIBGNAT_TARGET_PAIRS = $(LIBGNAT_TARGET_PAIRS_64)
> +      LIBGNAT_TARGET_PAIRS = \
> +      $(LIBGNAT_TARGET_PAIRS_COMMON) $(LIBGNAT_TARGET_PAIRS_64)
>      else
> -      LIBGNAT_TARGET_PAIRS = $(LIBGNAT_TARGET_PAIRS_32)
> +      LIBGNAT_TARGET_PAIRS = \
> +      $(LIBGNAT_TARGET_PAIRS_COMMON) $(LIBGNAT_TARGET_PAIRS_32)
>      endif
>    else
>      ifeq ($(strip $(MULTISUBDIR)),/sparcv7)
> -      LIBGNAT_TARGET_PAIRS = $(LIBGNAT_TARGET_PAIRS_32)
> +      LIBGNAT_TARGET_PAIRS = \
> +      $(LIBGNAT_TARGET_PAIRS_COMMON) $(LIBGNAT_TARGET_PAIRS_32)
>      else
> -      LIBGNAT_TARGET_PAIRS = $(LIBGNAT_TARGET_PAIRS_64)
> +      LIBGNAT_TARGET_PAIRS = \
> +      $(LIBGNAT_TARGET_PAIRS_COMMON) $(LIBGNAT_TARGET_PAIRS_64)
>      endif
>    endif

I think it's considerably clearer to use the += idiom found e.g. in

	http://gcc.gnu.org/ml/gcc-patches/2008-10/msg00026.html

That patch is fully tested now, but I didn't yet get around to committing it
because I wanted to produce a test case for GNAT.Byte_Swapping before using
g-bytswa-x86.adb for Solaris/x86_64.

	Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Faculty of Technology, Bielefeld University


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]