List:Commits« Previous MessageNext Message »
From:Mikael Ronstrom Date:November 20 2009 5:54pm
Subject:bzr commit into mysql-5.6-next-mr branch (mikael:2924) WL#5105
View as plain text  
#At file:///home/mikael/mysql_clones/mysql-next-mr-wl5105/ based on revid:davi.arnaut@stripped

 2924 Mikael Ronstrom	2009-11-20 [merge]
      WL#5105, Optimize ptr_cmp on Solaris by using memcmp instead of native ptr_cmp

    modified:
      mysys/ptr_cmp.c
=== modified file 'mysys/ptr_cmp.c'
--- a/mysys/ptr_cmp.c	2007-05-10 09:59:39 +0000
+++ b/mysys/ptr_cmp.c	2009-09-23 13:15:35 +0000
@@ -22,16 +22,39 @@
 #include "mysys_priv.h"
 #include <myisampack.h>
 
+#ifdef UNIV_SOLARIS
+/*
+ * On Solaris, memcmp() is normally faster than the unrolled ptr_compare_N
+ * functions, as memcmp() is usually a platform-specific implementation
+ * written in assembler, provided in /usr/lib/libc/libc_hwcap*.so.1.
+ * This implementation is also usually faster than the built-in memcmp
+ * supplied by GCC, so it is recommended to build with "-fno-builtin-memcmp"
+ * in CFLAGS if building with GCC on Solaris.
+ */
+
+#include <string.h>
+
+static int native_compare(size_t *length, unsigned char **a, unsigned char **b)
+{
+  return memcmp(*a, *b, *length);
+}
+
+#else	/* UNIV_SOLARIS */
+
 static int ptr_compare(size_t *compare_length, uchar **a, uchar **b);
 static int ptr_compare_0(size_t *compare_length, uchar **a, uchar **b);
 static int ptr_compare_1(size_t *compare_length, uchar **a, uchar **b);
 static int ptr_compare_2(size_t *compare_length, uchar **a, uchar **b);
 static int ptr_compare_3(size_t *compare_length, uchar **a, uchar **b);
+#endif	/* UNIV_SOLARIS */
 
 	/* Get a pointer to a optimal byte-compare function for a given size */
 
 qsort2_cmp get_ptr_compare (size_t size)
 {
+#ifdef UNIV_SOLARIS
+  return (qsort2_cmp) native_compare;
+#else
   if (size < 4)
     return (qsort2_cmp) ptr_compare;
   switch (size & 3) {
@@ -41,6 +64,7 @@ qsort2_cmp get_ptr_compare (size_t size)
     case 3: return (qsort2_cmp) ptr_compare_3;
     }
   return 0;					/* Impossible */
+#endif /* UNIV_SOLARIS */
 }
 
 


Attachment: [text/bzr-bundle] bzr/mikael@mysql.com-20091120175356-1d9vzjed701zfv3u.bundle
Thread
bzr commit into mysql-5.6-next-mr branch (mikael:2924) WL#5105Mikael Ronstrom20 Nov