annotate src/share/vm/utilities/quickSort.cpp @ 3779:04760e41b01e

7016112: CMS: crash during promotion testing Summary: Also reviewed by mikael.gerdin@oracle.com; stdlib:qsort() does byte-by-byte swapping on Windows. This leads to pointer shearing. Fix is to implement a quicksort that does full pointer updates. Reviewed-by: never, coleenp, ysr
author brutisso
date Tue, 28 Jun 2011 14:23:27 +0200
parents
children 8ab2f4108d20
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
3779
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
1 /*
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
2 * Copyright (c) 2011, Oracle and/or its affiliates. All rights reserved.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
4 *
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
7 * published by the Free Software Foundation.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
8 *
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
13 * accompanied this code).
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
14 *
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
18 *
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
21 * questions.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
22 *
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
23 */
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
24
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
25 #include "precompiled.hpp"
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
26 #include "utilities/quickSort.hpp"
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
27
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
28 #ifndef PRODUCT
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
29
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
30 // Unit tests
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
31
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
32 #include "runtime/os.hpp"
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
33 #include <stdlib.h>
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
34
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
35 static int test_comparator(int a, int b) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
36 if (a == b) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
37 return 0;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
38 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
39 if (a < b) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
40 return -1;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
41 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
42 return 1;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
43 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
44
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
45 static int test_even_odd_comparator(int a, int b) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
46 bool a_is_odd = (a % 2) == 1;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
47 bool b_is_odd = (b % 2) == 1;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
48 if (a_is_odd == b_is_odd) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
49 return 0;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
50 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
51 if (a_is_odd) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
52 return -1;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
53 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
54 return 1;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
55 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
56
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
57 static int test_stdlib_comparator(const void* a, const void* b) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
58 int ai = *(int*)a;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
59 int bi = *(int*)b;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
60 if (ai == bi) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
61 return 0;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
62 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
63 if (ai < bi) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
64 return -1;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
65 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
66 return 1;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
67 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
68
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
69 void QuickSort::print_array(const char* prefix, int* array, int length) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
70 tty->print("%s:", prefix);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
71 for (int i = 0; i < length; i++) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
72 tty->print(" %d", array[i]);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
73 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
74 tty->print_cr("");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
75 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
76
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
77 bool QuickSort::compare_arrays(int* actual, int* expected, int length) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
78 for (int i = 0; i < length; i++) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
79 if (actual[i] != expected[i]) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
80 print_array("Sorted array ", actual, length);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
81 print_array("Expected array", expected, length);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
82 return false;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
83 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
84 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
85 return true;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
86 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
87
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
88 template <class C>
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
89 bool QuickSort::sort_and_compare(int* arrayToSort, int* expectedResult, int length, C comparator, bool idempotent) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
90 sort<int, C>(arrayToSort, length, comparator, idempotent);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
91 return compare_arrays(arrayToSort, expectedResult, length);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
92 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
93
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
94 bool QuickSort::test_quick_sort() {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
95 tty->print_cr("test_quick_sort\n");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
96 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
97 int* test_array = NULL;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
98 int* expected_array = NULL;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
99 assert(sort_and_compare(test_array, expected_array, 0, test_comparator), "Empty array not handled");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
100 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
101 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
102 int test_array[] = {3};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
103 int expected_array[] = {3};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
104 assert(sort_and_compare(test_array, expected_array, 1, test_comparator), "Single value array not handled");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
105 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
106 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
107 int test_array[] = {3,2};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
108 int expected_array[] = {2,3};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
109 assert(sort_and_compare(test_array, expected_array, 2, test_comparator), "Array with 2 values not correctly sorted");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
110 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
111 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
112 int test_array[] = {3,2,1};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
113 int expected_array[] = {1,2,3};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
114 assert(sort_and_compare(test_array, expected_array, 3, test_comparator), "Array with 3 values not correctly sorted");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
115 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
116 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
117 int test_array[] = {4,3,2,1};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
118 int expected_array[] = {1,2,3,4};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
119 assert(sort_and_compare(test_array, expected_array, 4, test_comparator), "Array with 4 values not correctly sorted");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
120 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
121 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
122 int test_array[] = {7,1,5,3,6,9,8,2,4,0};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
123 int expected_array[] = {0,1,2,3,4,5,6,7,8,9};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
124 assert(sort_and_compare(test_array, expected_array, 10, test_comparator), "Array with 10 values not correctly sorted");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
125 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
126 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
127 int test_array[] = {4,4,1,4};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
128 int expected_array[] = {1,4,4,4};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
129 assert(sort_and_compare(test_array, expected_array, 4, test_comparator), "3 duplicates not sorted correctly");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
130 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
131 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
132 int test_array[] = {0,1,2,3,4,5,6,7,8,9};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
133 int expected_array[] = {0,1,2,3,4,5,6,7,8,9};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
134 assert(sort_and_compare(test_array, expected_array, 10, test_comparator), "Already sorted array not correctly sorted");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
135 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
136 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
137 // one of the random arrays that found an issue in the partion method.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
138 int test_array[] = {76,46,81,8,64,56,75,11,51,55,11,71,59,27,9,64,69,75,21,25,39,40,44,32,7,8,40,41,24,78,24,74,9,65,28,6,40,31,22,13,27,82};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
139 int expected_array[] = {6,7,8,8,9,9,11,11,13,21,22,24,24,25,27,27,28,31,32,39,40,40,40,41,44,46,51,55,56,59,64,64,65,69,71,74,75,75,76,78,81,82};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
140 assert(sort_and_compare(test_array, expected_array, 42, test_comparator), "Not correctly sorted");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
141 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
142 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
143 int test_array[] = {2,8,1,4};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
144 int expected_array[] = {1,4,2,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
145 assert(sort_and_compare(test_array, expected_array, 4, test_even_odd_comparator), "Even/odd not sorted correctly");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
146 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
147 { // Some idempotent tests
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
148 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
149 // An array of lenght 3 is only sorted by find_pivot. Make sure that it is idempotent.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
150 int test_array[] = {1,4,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
151 int expected_array[] = {1,4,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
152 assert(sort_and_compare(test_array, expected_array, 3, test_even_odd_comparator, true), "Even/odd not idempotent");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
153 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
154 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
155 int test_array[] = {1,7,9,4,8,2};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
156 int expected_array[] = {1,7,9,4,8,2};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
157 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
158 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
159 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
160 int test_array[] = {1,9,7,4,2,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
161 int expected_array[] = {1,9,7,4,2,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
162 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
163 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
164 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
165 int test_array[] = {7,9,1,2,8,4};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
166 int expected_array[] = {7,9,1,2,8,4};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
167 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
168 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
169 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
170 int test_array[] = {7,1,9,2,4,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
171 int expected_array[] = {7,1,9,2,4,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
172 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
173 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
174 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
175 int test_array[] = {9,1,7,4,8,2};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
176 int expected_array[] = {9,1,7,4,8,2};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
177 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
178 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
179 {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
180 int test_array[] = {9,7,1,4,2,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
181 int expected_array[] = {9,7,1,4,2,8};
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
182 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
183 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
184 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
185
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
186 // test sorting random arrays
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
187 for (int i = 0; i < 1000; i++) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
188 int length = os::random() % 100;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
189 int* test_array = new int[length];
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
190 int* expected_array = new int[length];
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
191 for (int j = 0; j < length; j++) {
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
192 // Choose random values, but get a chance of getting duplicates
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
193 test_array[j] = os::random() % (length * 2);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
194 expected_array[j] = test_array[j];
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
195 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
196
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
197 // Compare sorting to stdlib::qsort()
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
198 qsort(expected_array, length, sizeof(int), test_stdlib_comparator);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
199 assert(sort_and_compare(test_array, expected_array, length, test_comparator), "Random array not correctly sorted");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
200
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
201 // Make sure sorting is idempotent.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
202 // Both test_array and expected_array are sorted by the test_comparator.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
203 // Now sort them once with the test_even_odd_comparator. Then sort the
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
204 // test_array one more time with test_even_odd_comparator and verify that
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
205 // it is idempotent.
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
206 sort(expected_array, length, test_even_odd_comparator, true);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
207 sort(test_array, length, test_even_odd_comparator, true);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
208 assert(compare_arrays(test_array, expected_array, length), "Sorting identical arrays rendered different results");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
209 sort(test_array, length, test_even_odd_comparator, true);
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
210 assert(compare_arrays(test_array, expected_array, length), "Sorting already sorted array changed order of elements - not idempotent");
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
211
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
212 delete[] test_array;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
213 delete[] expected_array;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
214 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
215 return true;
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
216 }
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
217
04760e41b01e 7016112: CMS: crash during promotion testing
brutisso
parents:
diff changeset
218 #endif