93dc2d
commit 7de01e60c200c431d3469deb784da8fd4508fc15
93dc2d
Author: Florian Weimer <fweimer@redhat.com>
93dc2d
Date:   Fri Jan 14 20:16:05 2022 +0100
93dc2d
93dc2d
    elf/Makefile: Reflow and sort most variable assignments
93dc2d
93dc2d
    Reviewed-by: H.J. Lu <hjl.tools@gmail.com>
93dc2d
93dc2d
Conflicts:
93dc2d
	elf/Makefile
93dc2d
	  (Usual backporting differences.  LLD support is missing
93dc2d
	  downstream.)
93dc2d
93dc2d
diff --git a/elf/Makefile b/elf/Makefile
93dc2d
index 5b9c36bc6f0a3ee5..124905f96c88ab53 100644
93dc2d
--- a/elf/Makefile
93dc2d
+++ b/elf/Makefile
93dc2d
@@ -21,21 +21,62 @@ subdir		:= elf
93dc2d
 
93dc2d
 include ../Makeconfig
93dc2d
 
93dc2d
-headers		= elf.h bits/elfclass.h link.h bits/link.h
93dc2d
-routines	= $(all-dl-routines) dl-support dl-iteratephdr \
93dc2d
-		  dl-addr dl-addr-obj enbl-secure dl-profstub \
93dc2d
-		  dl-origin dl-libc dl-sym dl-sysdep dl-error \
93dc2d
-		  dl-reloc-static-pie libc_early_init rtld_static_init
93dc2d
+headers = \
93dc2d
+  bits/elfclass.h \
93dc2d
+  bits/link.h \
93dc2d
+  elf.h \
93dc2d
+  link.h \
93dc2d
+  # headers
93dc2d
+
93dc2d
+routines = \
93dc2d
+  $(all-dl-routines) \
93dc2d
+  dl-addr \
93dc2d
+  dl-addr-obj \
93dc2d
+  dl-error \
93dc2d
+  dl-iteratephdr \
93dc2d
+  dl-libc \
93dc2d
+  dl-origin \
93dc2d
+  dl-profstub \
93dc2d
+  dl-reloc-static-pie \
93dc2d
+  dl-support \
93dc2d
+  dl-sym \
93dc2d
+  dl-sysdep \
93dc2d
+  enbl-secure \
93dc2d
+  libc_early_init \
93dc2d
+  rtld_static_init \
93dc2d
+  # routines
93dc2d
 
93dc2d
 # The core dynamic linking functions are in libc for the static and
93dc2d
 # profiled libraries.
93dc2d
-dl-routines	= $(addprefix dl-,load lookup object reloc deps \
93dc2d
-				  runtime init fini debug misc \
93dc2d
-				  version profile tls origin scope \
93dc2d
-				  execstack open close trampoline \
93dc2d
-				  exception sort-maps lookup-direct \
93dc2d
-				  call-libc-early-init write \
93dc2d
-				  thread_gscope_wait tls_init_tp)
93dc2d
+dl-routines = \
93dc2d
+  dl-call-libc-early-init \
93dc2d
+  dl-close \
93dc2d
+  dl-debug \
93dc2d
+  dl-deps \
93dc2d
+  dl-exception \
93dc2d
+  dl-execstack \
93dc2d
+  dl-fini \
93dc2d
+  dl-init \
93dc2d
+  dl-load \
93dc2d
+  dl-lookup \
93dc2d
+  dl-lookup-direct \
93dc2d
+  dl-misc \
93dc2d
+  dl-object \
93dc2d
+  dl-open \
93dc2d
+  dl-origin \
93dc2d
+  dl-profile \
93dc2d
+  dl-reloc \
93dc2d
+  dl-runtime \
93dc2d
+  dl-scope \
93dc2d
+  dl-sort-maps \
93dc2d
+  dl-thread_gscope_wait \
93dc2d
+  dl-tls \
93dc2d
+  dl-tls_init_tp \
93dc2d
+  dl-trampoline \
93dc2d
+  dl-version \
93dc2d
+  dl-write \
93dc2d
+  # dl-routines
93dc2d
+
93dc2d
 ifeq (yes,$(use-ldconfig))
93dc2d
 dl-routines += dl-cache
93dc2d
 endif
93dc2d
@@ -58,16 +99,38 @@ endif
93dc2d
 
93dc2d
 all-dl-routines = $(dl-routines) $(sysdep-dl-routines)
93dc2d
 # But they are absent from the shared libc, because that code is in ld.so.
93dc2d
-elide-routines.os = $(all-dl-routines) dl-support enbl-secure dl-origin \
93dc2d
-		    dl-sysdep dl-exception dl-reloc-static-pie \
93dc2d
-		    thread_gscope_wait rtld_static_init
93dc2d
+elide-routines.os = \
93dc2d
+  $(all-dl-routines) \
93dc2d
+  dl-exception \
93dc2d
+  dl-origin \
93dc2d
+  dl-reloc-static-pie \
93dc2d
+  dl-support \
93dc2d
+  dl-sysdep \
93dc2d
+  enbl-secure \
93dc2d
+  rtld_static_init \
93dc2d
+  thread_gscope_wait \
93dc2d
+  # elide-routines.os
93dc2d
 
93dc2d
 # ld.so uses those routines, plus some special stuff for being the program
93dc2d
 # interpreter and operating independent of libc.
93dc2d
-rtld-routines	= rtld $(all-dl-routines) dl-sysdep dl-environ dl-minimal \
93dc2d
-  dl-error-minimal dl-conflict dl-hwcaps dl-hwcaps_split dl-hwcaps-subdirs \
93dc2d
-  dl-usage dl-diagnostics dl-diagnostics-kernel dl-diagnostics-cpu \
93dc2d
-  dl-mutex
93dc2d
+rtld-routines = \
93dc2d
+  $(all-dl-routines) \
93dc2d
+  dl-conflict \
93dc2d
+  dl-diagnostics \
93dc2d
+  dl-diagnostics-cpu \
93dc2d
+  dl-diagnostics-kernel \
93dc2d
+  dl-environ \
93dc2d
+  dl-error-minimal \
93dc2d
+  dl-hwcaps \
93dc2d
+  dl-hwcaps-subdirs \
93dc2d
+  dl-hwcaps_split \
93dc2d
+  dl-minimal \
93dc2d
+  dl-mutex \
93dc2d
+  dl-sysdep \
93dc2d
+  dl-usage \
93dc2d
+  rtld \
93dc2d
+  # rtld-routines
93dc2d
+
93dc2d
 all-rtld-routines = $(rtld-routines) $(sysdep-rtld-routines)
93dc2d
 
93dc2d
 CFLAGS-dl-runtime.c += -fexceptions -fasynchronous-unwind-tables
93dc2d
@@ -102,8 +165,16 @@ ld-map		= $(common-objpfx)ld.map
93dc2d
 endif
93dc2d
 
93dc2d
 ifeq (yes,$(build-shared))
93dc2d
-extra-objs	= $(all-rtld-routines:%=%.os) sofini.os interp.os
93dc2d
-generated	+= librtld.os dl-allobjs.os ld.so ldd
93dc2d
+extra-objs = \
93dc2d
+  $(all-rtld-routines:%=%.os) \
93dc2d
+  sofini.os \
93dc2d
+  interp.os \
93dc2d
+  # extra-objs
93dc2d
+generated += \
93dc2d
+  dl-allobjs.os \
93dc2d
+  ld.so ldd \
93dc2d
+  librtld.os \
93dc2d
+  # generated
93dc2d
 install-others	= $(inst_rtlddir)/$(rtld-installed-name) $(inst_bindir)/ld.so
93dc2d
 install-bin-script = ldd
93dc2d
 endif
93dc2d
@@ -121,8 +192,15 @@ others-static	+= ldconfig
93dc2d
 others		+= ldconfig
93dc2d
 install-rootsbin += ldconfig
93dc2d
 
93dc2d
-ldconfig-modules := cache readlib xmalloc xstrdup chroot_canon static-stubs \
93dc2d
-  stringtable
93dc2d
+ldconfig-modules := \
93dc2d
+  cache \
93dc2d
+  chroot_canon \
93dc2d
+  readlib \
93dc2d
+  static-stubs \
93dc2d
+  stringtable \
93dc2d
+  xmalloc \
93dc2d
+  xstrdup \
93dc2d
+  # ldconfig-modules
93dc2d
 extra-objs	+= $(ldconfig-modules:=.o)
93dc2d
 others-extras   = $(ldconfig-modules)
93dc2d
 endif
93dc2d
@@ -156,23 +234,36 @@ $(inst_auditdir)/sotruss-lib.so: $(objpfx)sotruss-lib.so $(+force)
93dc2d
 	$(do-install-program)
93dc2d
 endif
93dc2d
 
93dc2d
-tests-static-normal := tst-array1-static tst-array5-static \
93dc2d
-	       tst-dl-iter-static \
93dc2d
-	       tst-tlsalign-static tst-tlsalign-extern-static \
93dc2d
-	       tst-linkall-static tst-env-setuid tst-env-setuid-tunables \
93dc2d
-	       tst-single_threaded-static tst-single_threaded-pthread-static \
93dc2d
-	       tst-dst-static tst-getauxval-static
93dc2d
-
93dc2d
-tests-static-internal := tst-tls1-static tst-tls2-static \
93dc2d
-	       tst-ptrguard1-static tst-stackguard1-static \
93dc2d
-	       tst-tls1-static-non-pie
93dc2d
+tests-static-normal := \
93dc2d
+  tst-array1-static \
93dc2d
+  tst-array5-static \
93dc2d
+  tst-dl-iter-static \
93dc2d
+  tst-dst-static \
93dc2d
+  tst-env-setuid \
93dc2d
+  tst-env-setuid-tunables \
93dc2d
+  tst-getauxval-static \
93dc2d
+  tst-linkall-static \
93dc2d
+  tst-single_threaded-pthread-static \
93dc2d
+  tst-single_threaded-static \
93dc2d
+  tst-tlsalign-extern-static \
93dc2d
+  tst-tlsalign-static \
93dc2d
+  # tests-static-normal
93dc2d
+
93dc2d
+tests-static-internal := \
93dc2d
+  tst-ptrguard1-static \
93dc2d
+  tst-stackguard1-static \
93dc2d
+  tst-tls1-static \
93dc2d
+  tst-tls2-static \
93dc2d
+  tst-tls1-static-non-pie \
93dc2d
+  # tests-static-internal
93dc2d
 
93dc2d
 CRT-tst-tls1-static-non-pie := $(csu-objpfx)crt1.o
93dc2d
 tst-tls1-static-non-pie-no-pie = yes
93dc2d
 
93dc2d
 tests-container := \
93dc2d
-			  tst-ldconfig-bad-aux-cache \
93dc2d
-			  tst-ldconfig-ld_so_conf-update
93dc2d
+  tst-ldconfig-bad-aux-cache \
93dc2d
+  tst-ldconfig-ld_so_conf-update \
93dc2d
+  # tests-container
93dc2d
 
93dc2d
 ifeq (no,$(build-hardcoded-path-in-tests))
93dc2d
 # This is an ld.so.cache test, and RPATH/RUNPATH in the executable
93dc2d
@@ -180,14 +271,32 @@ ifeq (no,$(build-hardcoded-path-in-tests))
93dc2d
 tests-container += tst-glibc-hwcaps-prepend-cache
93dc2d
 endif
93dc2d
 
93dc2d
-tests := tst-tls9 tst-leaks1 \
93dc2d
-	tst-array1 tst-array2 tst-array3 tst-array4 tst-array5 \
93dc2d
-	tst-auxv tst-stringtable
93dc2d
-tests-internal := tst-tls1 tst-tls2 $(tests-static-internal)
93dc2d
+tests := \
93dc2d
+  tst-array1 \
93dc2d
+  tst-array2 \
93dc2d
+  tst-array3 \
93dc2d
+  tst-array4 \
93dc2d
+  tst-array5 \
93dc2d
+  tst-auxv \
93dc2d
+  tst-leaks1 \
93dc2d
+  tst-stringtable \
93dc2d
+  tst-tls9 \
93dc2d
+  # tests
93dc2d
+
93dc2d
+tests-internal := \
93dc2d
+  $(tests-static-internal) \
93dc2d
+  tst-tls1 \
93dc2d
+  tst-tls2 \
93dc2d
+  # tests-internal
93dc2d
+
93dc2d
 tests-static := $(tests-static-normal) $(tests-static-internal)
93dc2d
 
93dc2d
 ifeq (yes,$(build-shared))
93dc2d
-tests-static += tst-tls9-static tst-single_threaded-static-dlopen
93dc2d
+tests-static += \
93dc2d
+  tst-single_threaded-static-dlopen \
93dc2d
+  tst-tls9-static \
93dc2d
+  # tests-static
93dc2d
+
93dc2d
 static-dlopen-environment = \
93dc2d
   LD_LIBRARY_PATH=$(ld-library-path):$(common-objpfx)dlfcn
93dc2d
 tst-tls9-static-ENV = $(static-dlopen-environment)
93dc2d
@@ -313,33 +422,69 @@ tests += \
93dc2d
   unload6 \
93dc2d
   unload7 \
93dc2d
   unload8 \
93dc2d
-#	 reldep9
93dc2d
+  # tests
93dc2d
 tests-cxx = \
93dc2d
   tst-dlopen-nodelete-reloc \
93dc2d
   tst-nodelete \
93dc2d
   tst-unique3 \
93dc2d
   tst-unique4 \
93dc2d
-# tests-cxx
93dc2d
+  # tests-cxx
93dc2d
 
93dc2d
 tests += $(if $(CXX),$(tests-cxx))
93dc2d
-tests-internal += loadtest unload unload2 circleload1 \
93dc2d
-	 neededtest neededtest2 neededtest3 neededtest4 \
93dc2d
-	 tst-tls3 tst-tls6 tst-tls7 tst-tls8 tst-dlmopen2 \
93dc2d
-	 tst-ptrguard1 tst-stackguard1 \
93dc2d
-	 tst-create_format1 tst-tls-surplus tst-dl-hwcaps_split
93dc2d
-tests-container += tst-pldd tst-dlopen-tlsmodid-container \
93dc2d
-  tst-dlopen-self-container tst-preload-pthread-libc
93dc2d
-test-srcs = tst-pathopt
93dc2d
+
93dc2d
+tests-internal += \
93dc2d
+  circleload1 \
93dc2d
+  loadtest \
93dc2d
+  neededtest \
93dc2d
+  neededtest2 \
93dc2d
+  neededtest3 \
93dc2d
+  neededtest4 \
93dc2d
+  tst-create_format1 \
93dc2d
+  tst-dl-hwcaps_split \
93dc2d
+  tst-dlmopen2 \
93dc2d
+  tst-ptrguard1 \
93dc2d
+  tst-stackguard1 \
93dc2d
+  tst-tls-surplus \
93dc2d
+  tst-tls3 \
93dc2d
+  tst-tls6 \
93dc2d
+  tst-tls7 \
93dc2d
+  tst-tls8 \
93dc2d
+  unload \
93dc2d
+  unload2 \
93dc2d
+  # tests-internal
93dc2d
+
93dc2d
+tests-container += \
93dc2d
+  tst-dlopen-self-container \
93dc2d
+  tst-dlopen-tlsmodid-container \
93dc2d
+  tst-pldd \
93dc2d
+  tst-preload-pthread-libc \
93dc2d
+  # tests-container
93dc2d
+
93dc2d
+test-srcs = \
93dc2d
+  tst-pathopt \
93dc2d
+  # tests-srcs
93dc2d
+
93dc2d
+ifeq (yes,$(have-fpie))
93dc2d
+tests-pie += tst-align3
93dc2d
+endif
93dc2d
 selinux-enabled := $(shell cat /selinux/enforce 2> /dev/null)
93dc2d
+
93dc2d
 ifneq ($(selinux-enabled),1)
93dc2d
-tests-execstack-yes = tst-execstack tst-execstack-needed tst-execstack-prog
93dc2d
+tests-execstack-yes = \
93dc2d
+  tst-execstack \
93dc2d
+  tst-execstack-needed \
93dc2d
+  tst-execstack-prog \
93dc2d
+  # tests-execstack-yes
93dc2d
 endif
93dc2d
 endif
93dc2d
 tests += $(tests-execstack-$(have-z-execstack))
93dc2d
 ifeq ($(run-built-tests),yes)
93dc2d
-tests-special += $(objpfx)tst-leaks1-mem.out \
93dc2d
-		 $(objpfx)noload-mem.out \
93dc2d
-		 $(objpfx)tst-ldconfig-X.out $(objpfx)tst-rtld-help.out
93dc2d
+tests-special += \
93dc2d
+  $(objpfx)noload-mem.out \
93dc2d
+  $(objpfx)tst-ldconfig-X.out \
93dc2d
+  $(objpfx)tst-leaks1-mem.out \
93dc2d
+  $(objpfx)tst-rtld-help.out \
93dc2d
+  # tests-special
93dc2d
 endif
93dc2d
 tlsmod17a-suffixes = 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19
93dc2d
 tlsmod18a-suffixes = 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19
93dc2d
@@ -356,9 +501,16 @@ tst-tls-many-dynamic-modules-dep = \
93dc2d
 tst-tls-many-dynamic-modules-dep-bad-suffixes = 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14
93dc2d
 tst-tls-many-dynamic-modules-dep-bad = \
93dc2d
   $(foreach n,$(tst-tls-many-dynamic-modules-dep-bad-suffixes),tst-tls-manydynamic$(n)mod-dep-bad)
93dc2d
-extra-test-objs += $(tlsmod17a-modules:=.os) $(tlsmod18a-modules:=.os) \
93dc2d
-		   tst-tlsalign-vars.o
93dc2d
-test-extras += tst-tlsmod17a tst-tlsmod18a tst-tlsalign-vars
93dc2d
+extra-test-objs += \
93dc2d
+  $(tlsmod17a-modules:=.os) \
93dc2d
+  $(tlsmod18a-modules:=.os) \
93dc2d
+  tst-tlsalign-vars.o \
93dc2d
+  # extra-test-objs
93dc2d
+test-extras += \
93dc2d
+  tst-tlsalign-vars \
93dc2d
+  tst-tlsmod17a \
93dc2d
+  tst-tlsmod18a \
93dc2d
+  # test-extras
93dc2d
 modules-names = \
93dc2d
   circlemod1 \
93dc2d
   circlemod1a \
93dc2d
@@ -610,17 +762,17 @@ modules-names-cxx = \
93dc2d
   tst-unique3lib \
93dc2d
   tst-unique3lib2 \
93dc2d
   tst-unique4lib \
93dc2d
-# modules-names-cxx
93dc2d
+  # modules-names-cxx
93dc2d
 
93dc2d
 modules-names += \
93dc2d
   $(if $(CXX),$(modules-names-cxx)) \
93dc2d
   $(modules-execstack-$(have-z-execstack)) \
93dc2d
+  $(tlsmod17a-modules) \
93dc2d
+  $(tlsmod18a-modules) \
93dc2d
   $(tst-tls-many-dynamic-modules) \
93dc2d
   $(tst-tls-many-dynamic-modules-dep) \
93dc2d
   $(tst-tls-many-dynamic-modules-dep-bad) \
93dc2d
-  $(tlsmod17a-modules) \
93dc2d
-  $(tlsmod18a-modules) \
93dc2d
-# modules-names
93dc2d
+  # modules-names
93dc2d
 
93dc2d
 # Most modules build with _ISOMAC defined, but those filtered out
93dc2d
 # depend on internal headers.
93dc2d
@@ -669,35 +821,70 @@ modules-names-nobuild := filtmod1 tst-big-note-lib tst-ro-dynamic-mod
93dc2d
 tests += $(tests-static)
93dc2d
 
93dc2d
 ifeq (yes,$(have-ifunc))
93dc2d
-tests-ifuncstatic := ifuncmain1static ifuncmain1picstatic \
93dc2d
-		ifuncmain2static ifuncmain2picstatic \
93dc2d
-		ifuncmain4static ifuncmain4picstatic \
93dc2d
-		ifuncmain5static ifuncmain5picstatic \
93dc2d
-		ifuncmain7static ifuncmain7picstatic
93dc2d
+tests-ifuncstatic := \
93dc2d
+  ifuncmain1static \
93dc2d
+  ifuncmain1picstatic \
93dc2d
+  ifuncmain2static \
93dc2d
+  ifuncmain2picstatic \
93dc2d
+  ifuncmain4static \
93dc2d
+  ifuncmain4picstatic \
93dc2d
+  ifuncmain5static \
93dc2d
+  ifuncmain5picstatic \
93dc2d
+  ifuncmain7static \
93dc2d
+  ifuncmain7picstatic \
93dc2d
+  # tests-ifuncstatic
93dc2d
 ifeq (yes,$(have-gcc-ifunc))
93dc2d
 tests-ifuncstatic += ifuncmain9static ifuncmain9picstatic
93dc2d
 endif
93dc2d
 tests-static += $(tests-ifuncstatic)
93dc2d
 tests-internal += $(tests-ifuncstatic)
93dc2d
 ifeq (yes,$(build-shared))
93dc2d
-tests += tst-ifunc-fault-lazy tst-ifunc-fault-bindnow
93dc2d
+tests += \
93dc2d
+  tst-ifunc-fault-bindnow \
93dc2d
+  tst-ifunc-fault-lazy \
93dc2d
+  # tests
93dc2d
 # Note: sysdeps/x86_64/ifuncmain8.c uses ifuncmain8.
93dc2d
 tests-internal += \
93dc2d
-	 ifuncmain1 ifuncmain1pic ifuncmain1vis ifuncmain1vispic \
93dc2d
-	 ifuncmain1staticpic \
93dc2d
-	 ifuncmain2 ifuncmain2pic ifuncmain3 ifuncmain4 \
93dc2d
-	 ifuncmain5 ifuncmain5pic ifuncmain5staticpic \
93dc2d
-	 ifuncmain7 ifuncmain7pic
93dc2d
+  ifuncmain1 \
93dc2d
+  ifuncmain1pic \
93dc2d
+  ifuncmain1staticpic \
93dc2d
+  ifuncmain1vis \
93dc2d
+  ifuncmain1vispic \
93dc2d
+  ifuncmain2 \
93dc2d
+  ifuncmain2pic \
93dc2d
+  ifuncmain3 \
93dc2d
+  ifuncmain4 \
93dc2d
+  ifuncmain5 \
93dc2d
+  ifuncmain5pic \
93dc2d
+  ifuncmain5staticpic \
93dc2d
+  ifuncmain7 \
93dc2d
+  ifuncmain7pic \
93dc2d
+  # tests-internal
93dc2d
 ifeq (yes,$(have-gcc-ifunc))
93dc2d
-tests-internal += ifuncmain9 ifuncmain9pic
93dc2d
+tests-internal += \
93dc2d
+  ifuncmain9 \
93dc2d
+  ifuncmain9pic \
93dc2d
+  # tests-internal
93dc2d
 endif
93dc2d
-ifunc-test-modules = ifuncdep1 ifuncdep1pic ifuncdep2 ifuncdep2pic \
93dc2d
-		     ifuncdep5 ifuncdep5pic
93dc2d
+ifunc-test-modules = \
93dc2d
+  ifuncdep1 \
93dc2d
+  ifuncdep1pic \
93dc2d
+  ifuncdep2 \
93dc2d
+  ifuncdep2pic \
93dc2d
+  ifuncdep5 \
93dc2d
+  ifuncdep5pic \
93dc2d
+  # ifunc-test-modules
93dc2d
 extra-test-objs += $(ifunc-test-modules:=.o)
93dc2d
 test-internal-extras += $(ifunc-test-modules)
93dc2d
 ifeq (yes,$(have-fpie))
93dc2d
-ifunc-pie-tests = ifuncmain1pie ifuncmain1vispie ifuncmain1staticpie \
93dc2d
-		  ifuncmain5pie ifuncmain6pie ifuncmain7pie
93dc2d
+ifunc-pie-tests = \
93dc2d
+  ifuncmain1pie \
93dc2d
+  ifuncmain1staticpie \
93dc2d
+  ifuncmain1vispie \
93dc2d
+  ifuncmain5pie \
93dc2d
+  ifuncmain6pie \
93dc2d
+  ifuncmain7pie \
93dc2d
+  # ifunc-pie-tests
93dc2d
 ifeq (yes,$(have-gcc-ifunc))
93dc2d
 ifunc-pie-tests += ifuncmain9pie
93dc2d
 endif
93dc2d
@@ -707,30 +894,50 @@ endif
93dc2d
 tests-internal += $(ifunc-pie-tests)
93dc2d
 tests-pie += $(ifunc-pie-tests)
93dc2d
 endif
93dc2d
-modules-names += ifuncmod1 ifuncmod3 ifuncmod5 ifuncmod6
93dc2d
+modules-names += \
93dc2d
+  ifuncmod1 \
93dc2d
+  ifuncmod3 \
93dc2d
+  ifuncmod5 \
93dc2d
+  ifuncmod6 \
93dc2d
+  # module-names
93dc2d
 endif
93dc2d
 endif
93dc2d
 
93dc2d
 ifeq (yes,$(build-shared))
93dc2d
 ifeq ($(run-built-tests),yes)
93dc2d
-tests-special += $(objpfx)tst-pathopt.out $(objpfx)tst-rtld-load-self.out \
93dc2d
-		 $(objpfx)tst-rtld-preload.out $(objpfx)argv0test.out \
93dc2d
-		 $(objpfx)tst-rtld-help.out
93dc2d
+tests-special += \
93dc2d
+  $(objpfx)argv0test.out \
93dc2d
+  $(objpfx)tst-pathopt.out \
93dc2d
+  $(objpfx)tst-rtld-help.out \
93dc2d
+  $(objpfx)tst-rtld-load-self.out \
93dc2d
+  $(objpfx)tst-rtld-preload.out \
93dc2d
+  # tests-special
93dc2d
 endif
93dc2d
-tests-special += $(objpfx)check-textrel.out $(objpfx)check-execstack.out \
93dc2d
-		 $(objpfx)check-wx-segment.out \
93dc2d
-		 $(objpfx)check-localplt.out $(objpfx)check-initfini.out
93dc2d
+tests-special += \
93dc2d
+  $(objpfx)check-execstack.out \
93dc2d
+  $(objpfx)check-initfini.out \
93dc2d
+  $(objpfx)check-localplt.out \
93dc2d
+  $(objpfx)check-textrel.out \
93dc2d
+  $(objpfx)check-wx-segment.out \
93dc2d
+  # tests-special
93dc2d
 endif
93dc2d
 
93dc2d
 ifeq ($(run-built-tests),yes)
93dc2d
-tests-special += $(objpfx)order-cmp.out $(objpfx)tst-array1-cmp.out \
93dc2d
-		 $(objpfx)tst-array1-static-cmp.out \
93dc2d
-		 $(objpfx)tst-array2-cmp.out $(objpfx)tst-array3-cmp.out \
93dc2d
-		 $(objpfx)tst-array4-cmp.out $(objpfx)tst-array5-cmp.out \
93dc2d
-		 $(objpfx)tst-array5-static-cmp.out $(objpfx)order2-cmp.out \
93dc2d
-		 $(objpfx)tst-initorder-cmp.out \
93dc2d
-		 $(objpfx)tst-initorder2-cmp.out $(objpfx)tst-unused-dep.out \
93dc2d
-		 $(objpfx)tst-unused-dep-cmp.out
93dc2d
+tests-special += \
93dc2d
+  $(objpfx)order-cmp.out \
93dc2d
+  $(objpfx)order2-cmp.out \
93dc2d
+  $(objpfx)tst-array1-cmp.out \
93dc2d
+  $(objpfx)tst-array1-static-cmp.out \
93dc2d
+  $(objpfx)tst-array2-cmp.out \
93dc2d
+  $(objpfx)tst-array3-cmp.out \
93dc2d
+  $(objpfx)tst-array4-cmp.out \
93dc2d
+  $(objpfx)tst-array5-cmp.out \
93dc2d
+  $(objpfx)tst-array5-static-cmp.out \
93dc2d
+  $(objpfx)tst-initorder-cmp.out \
93dc2d
+  $(objpfx)tst-initorder2-cmp.out \
93dc2d
+  $(objpfx)tst-unused-dep-cmp.out \
93dc2d
+  $(objpfx)tst-unused-dep.out \
93dc2d
+  # tests-special
93dc2d
 endif
93dc2d
 
93dc2d
 ifndef avoid-generated
93dc2d
@@ -835,6 +1042,7 @@ rtld-stubbed-symbols = \
93dc2d
   free \
93dc2d
   malloc \
93dc2d
   realloc \
93dc2d
+  # rtld-stubbed-symbols
93dc2d
 
93dc2d
 ifeq ($(have-ssp),yes)
93dc2d
 # rtld is not built with the stack protector, so these references will