This is the mail archive of the libc-ports@sources.redhat.com mailing list for the libc-ports 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: [PATCH] tile: don't guard setting abi- variables with ifeq ($(subdir),misc)


On Wed, 30 May 2012, Chris Metcalf wrote:

> Joseph, you may want to check how mips is doing this, since I originally
> copied the "make" support from mips.  It worked fine to create syscall.h,
> but it was missing out on creating the multiple versions of stub.h,
> until I removed the $(subdir) guard, which presumably is somewhat pointless
> when we're just setting some variables anyway.  I note that the new arm
> support doesn't use a $(subdir) guard either.

Thanks for pointing this out; I've applied this patch.  I was following 
HJ's instructions, which didn't mention anything about moving things out 
of conditionals.

diff --git a/ChangeLog.mips b/ChangeLog.mips
index ced2d36..6243cfc 100644
--- a/ChangeLog.mips
+++ b/ChangeLog.mips
@@ -1,5 +1,8 @@
 2012-05-30  Joseph Myers  <joseph@codesourcery.com>
 
+	* sysdeps/unix/sysv/linux/mips/Makefile [subdir = misc]: Move ABI
+	settings out of conditional.
+
 	* sysdeps/unix/sysv/linux/mips/syscalls.list: Remove
 	__connect_internal alias.
 
diff --git a/sysdeps/unix/sysv/linux/mips/Makefile b/sysdeps/unix/sysv/linux/mips/Makefile
index fd6e3e0..2bbc9a4 100644
--- a/sysdeps/unix/sysv/linux/mips/Makefile
+++ b/sysdeps/unix/sysv/linux/mips/Makefile
@@ -6,6 +6,7 @@ ifeq ($(subdir),misc)
 sysdep_routines += cachectl cacheflush sysmips _test_and_set
 
 sysdep_headers += sys/cachectl.h sys/sysmips.h sys/tas.h
+endif
 
 # _MIPS_SIM_ABI32 == 1, _MIPS_SIM_ABIN32 == 2, _MIPS_SIM_ABI64 == 3
 abi-variants := o32 n32 n64
@@ -16,7 +17,6 @@ abi-n32-options := -D_MIPS_SIM=2
 abi-n32-condition := _MIPS_SIM == _MIPS_SIM_ABIN32
 abi-n64-options := -D_MIPS_SIM=3
 abi-n64-condition := _MIPS_SIM == _MIPS_SIM_ABI64
-endif
 
 ifeq ($(subdir),elf)
 ifeq ($(build-shared),yes)

-- 
Joseph S. Myers
joseph@codesourcery.com


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