about summary refs log tree commit diff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2001-08-04 15:05:57 +0000
committerUlrich Drepper <drepper@redhat.com>2001-08-04 15:05:57 +0000
commitc2a714f32e6c56358f6dc9400de882753e966d22 (patch)
tree1b89401522a1b171f3f11ea77f41571fe126aefc
parent6906cea4cc14c72110dc35c625491d482d4caef1 (diff)
downloadglibc-c2a714f32e6c56358f6dc9400de882753e966d22.tar.gz
glibc-c2a714f32e6c56358f6dc9400de882753e966d22.tar.xz
glibc-c2a714f32e6c56358f6dc9400de882753e966d22.zip
Update.
	* elf/rtld.c: Correct use of __builtin_expect.

	* elf/dl-close.c: Remove commented-out code.
-rw-r--r--ChangeLog4
-rw-r--r--elf/dl-close.c3
-rw-r--r--elf/rtld.c2
3 files changed, 6 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index 923c79caa1..4f321a3ce1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2001-08-04  Ulrich Drepper  <drepper@redhat.com>
 
+	* elf/rtld.c: Correct use of __builtin_expect.
+
+	* elf/dl-close.c: Remove commented-out code.
+
 	* libio/genops.c (_IO_unbuffer_write): Force streams in narrow
 	orientation so that conversion modules aren't used anymore.
 	* malloc/set-freeres.c: Call _IO_cleanup before running subfreeres
diff --git a/elf/dl-close.c b/elf/dl-close.c
index 5ec8615a9a..3d24892d28 100644
--- a/elf/dl-close.c
+++ b/elf/dl-close.c
@@ -67,8 +67,7 @@ _dl_close (void *_map)
   if (map->l_opencount > 1 || map->l_type != lt_loaded)
     {
       /* There are still references to this object.  Do nothing more.  */
-      if (//__builtin_expect (_dl_debug_files, 0))
-	  __builtin_expect (_dl_debug_mask & DL_DEBUG_FILES, 0))
+      if (__builtin_expect (_dl_debug_mask & DL_DEBUG_FILES, 0))
 	{
 	  char buf[20];
 
diff --git a/elf/rtld.c b/elf/rtld.c
index 36dbc6f94c..2eddb5a40a 100644
--- a/elf/rtld.c
+++ b/elf/rtld.c
@@ -823,7 +823,7 @@ of this helper program; chances are you did not intend to run this program.\n\
   if (_dl_rtld_map.l_next)
     _dl_rtld_map.l_next->l_prev = _dl_rtld_map.l_prev;
 
-  if (__builtin_expect (_dl_rtld_map.l_opencount, 2) > 1)
+  if (__builtin_expect (_dl_rtld_map.l_opencount > 1, 1))
     {
       /* Some DT_NEEDED entry referred to the interpreter object itself, so
 	 put it back in the list of visible objects.  We insert it into the