Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 1 | //===- ListReducer.h - Trim down list while retaining property --*- C++ -*-===// |
Misha Brukman | 650ba8e | 2005-04-22 00:00:37 +0000 | [diff] [blame] | 2 | // |
Chandler Carruth | 2946cd7 | 2019-01-19 08:50:56 +0000 | [diff] [blame] | 3 | // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. |
| 4 | // See https://llvm.org/LICENSE.txt for license information. |
| 5 | // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception |
Misha Brukman | 650ba8e | 2005-04-22 00:00:37 +0000 | [diff] [blame] | 6 | // |
John Criswell | 09344dc | 2003-10-20 17:47:21 +0000 | [diff] [blame] | 7 | //===----------------------------------------------------------------------===// |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 8 | // |
| 9 | // This class is to be used as a base class for operations that want to zero in |
| 10 | // on a subset of the input which still causes the bug we are tracking. |
| 11 | // |
| 12 | //===----------------------------------------------------------------------===// |
| 13 | |
Benjamin Kramer | a7c40ef | 2014-08-13 16:26:38 +0000 | [diff] [blame] | 14 | #ifndef LLVM_TOOLS_BUGPOINT_LISTREDUCER_H |
| 15 | #define LLVM_TOOLS_BUGPOINT_LISTREDUCER_H |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 16 | |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 17 | #include "llvm/Support/Error.h" |
Chandler Carruth | 4d88a1c | 2012-12-04 10:44:52 +0000 | [diff] [blame] | 18 | #include "llvm/Support/raw_ostream.h" |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 19 | #include <algorithm> |
Chandler Carruth | 4d88a1c | 2012-12-04 10:44:52 +0000 | [diff] [blame] | 20 | #include <cstdlib> |
Marshall Clow | e9110d7 | 2017-02-16 14:37:03 +0000 | [diff] [blame] | 21 | #include <random> |
Chandler Carruth | 4d88a1c | 2012-12-04 10:44:52 +0000 | [diff] [blame] | 22 | #include <vector> |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 23 | |
Brian Gaeke | 960707c | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 24 | namespace llvm { |
| 25 | |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 26 | extern bool BugpointIsInterrupted; |
| 27 | |
| 28 | template <typename ElTy> struct ListReducer { |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 29 | enum TestResult { |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 30 | NoFailure, // No failure of the predicate was detected |
| 31 | KeepSuffix, // The suffix alone satisfies the predicate |
| 32 | KeepPrefix // The prefix alone satisfies the predicate |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 33 | }; |
| 34 | |
Chris Lattner | d7d574d | 2004-05-11 20:41:07 +0000 | [diff] [blame] | 35 | virtual ~ListReducer() {} |
| 36 | |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 37 | /// This virtual function should be overriden by subclasses to implement the |
| 38 | /// test desired. The testcase is only required to test to see if the Kept |
| 39 | /// list still satisfies the property, but if it is going to check the prefix |
| 40 | /// anyway, it can. |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 41 | virtual Expected<TestResult> doTest(std::vector<ElTy> &Prefix, |
| 42 | std::vector<ElTy> &Kept) = 0; |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 43 | |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 44 | /// This function attempts to reduce the length of the specified list while |
| 45 | /// still maintaining the "test" property. This is the core of the "work" |
| 46 | /// that bugpoint does. |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 47 | Expected<bool> reduceList(std::vector<ElTy> &TheList) { |
Misha Brukman | 87154fd | 2003-07-30 20:15:56 +0000 | [diff] [blame] | 48 | std::vector<ElTy> empty; |
Marshall Clow | e9110d7 | 2017-02-16 14:37:03 +0000 | [diff] [blame] | 49 | std::mt19937 randomness(0x6e5ea738); // Seed the random number generator |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 50 | Expected<TestResult> Result = doTest(TheList, empty); |
| 51 | if (Error E = Result.takeError()) |
Bill Wendling | c55cf4a | 2020-02-10 07:06:45 -0800 | [diff] [blame] | 52 | return std::move(E); |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 53 | switch (*Result) { |
Misha Brukman | 87154fd | 2003-07-30 20:15:56 +0000 | [diff] [blame] | 54 | case KeepPrefix: |
| 55 | if (TheList.size() == 1) // we are done, it's the base case and it fails |
| 56 | return true; |
Misha Brukman | 650ba8e | 2005-04-22 00:00:37 +0000 | [diff] [blame] | 57 | else |
Misha Brukman | 87154fd | 2003-07-30 20:15:56 +0000 | [diff] [blame] | 58 | break; // there's definitely an error, but we need to narrow it down |
| 59 | |
| 60 | case KeepSuffix: |
| 61 | // cannot be reached! |
Nick Lewycky | 6ba630b | 2010-04-12 05:08:25 +0000 | [diff] [blame] | 62 | llvm_unreachable("bugpoint ListReducer internal error: " |
| 63 | "selected empty set."); |
Misha Brukman | 87154fd | 2003-07-30 20:15:56 +0000 | [diff] [blame] | 64 | |
| 65 | case NoFailure: |
| 66 | return false; // there is no failure with the full set of passes/funcs! |
| 67 | } |
| 68 | |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 69 | // Maximal number of allowed splitting iterations, |
| 70 | // before the elements are randomly shuffled. |
| 71 | const unsigned MaxIterationsWithoutProgress = 3; |
Tobias Grosser | 56eab36 | 2015-07-26 15:18:45 +0000 | [diff] [blame] | 72 | |
| 73 | // Maximal number of allowed single-element trim iterations. We add a |
Simon Pilgrim | dae11f7 | 2016-11-20 13:31:13 +0000 | [diff] [blame] | 74 | // threshold here as single-element reductions may otherwise take a |
Tobias Grosser | 56eab36 | 2015-07-26 15:18:45 +0000 | [diff] [blame] | 75 | // very long time to complete. |
| 76 | const unsigned MaxTrimIterationsWithoutBackJump = 3; |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 77 | bool ShufflingEnabled = true; |
| 78 | |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 79 | Backjump: |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 80 | unsigned MidTop = TheList.size(); |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 81 | unsigned MaxIterations = MaxIterationsWithoutProgress; |
| 82 | unsigned NumOfIterationsWithoutProgress = 0; |
| 83 | while (MidTop > 1) { // Binary split reduction loop |
Chris Lattner | beb01fa | 2005-08-02 02:16:17 +0000 | [diff] [blame] | 84 | // Halt if the user presses ctrl-c. |
| 85 | if (BugpointIsInterrupted) { |
Dan Gohman | d8db376 | 2009-07-15 16:35:29 +0000 | [diff] [blame] | 86 | errs() << "\n\n*** Reduction Interrupted, cleaning up...\n\n"; |
Chris Lattner | beb01fa | 2005-08-02 02:16:17 +0000 | [diff] [blame] | 87 | return true; |
| 88 | } |
Bill Wendling | 8ff4320 | 2008-02-26 10:46:10 +0000 | [diff] [blame] | 89 | |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 90 | // If the loop doesn't make satisfying progress, try shuffling. |
| 91 | // The purpose of shuffling is to avoid the heavy tails of the |
| 92 | // distribution (improving the speed of convergence). |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 93 | if (ShufflingEnabled && NumOfIterationsWithoutProgress > MaxIterations) { |
Bill Wendling | 8ff4320 | 2008-02-26 10:46:10 +0000 | [diff] [blame] | 94 | std::vector<ElTy> ShuffledList(TheList); |
Marshall Clow | e9110d7 | 2017-02-16 14:37:03 +0000 | [diff] [blame] | 95 | std::shuffle(ShuffledList.begin(), ShuffledList.end(), randomness); |
Dan Gohman | d8db376 | 2009-07-15 16:35:29 +0000 | [diff] [blame] | 96 | errs() << "\n\n*** Testing shuffled set...\n\n"; |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 97 | // Check that random shuffle doesn't lose the bug |
| 98 | Expected<TestResult> Result = doTest(ShuffledList, empty); |
| 99 | // TODO: Previously, this error was ignored and we treated it as if |
| 100 | // shuffling hid the bug. This should really either be consumeError if |
| 101 | // that behaviour was sensible, or we should propagate the error. |
| 102 | assert(!Result.takeError() && "Shuffling caused internal error?"); |
| 103 | |
| 104 | if (*Result == KeepPrefix) { |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 105 | // If the bug is still here, use the shuffled list. |
| 106 | TheList.swap(ShuffledList); |
| 107 | MidTop = TheList.size(); |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 108 | // Must increase the shuffling treshold to avoid the small |
Simon Pilgrim | dae11f7 | 2016-11-20 13:31:13 +0000 | [diff] [blame] | 109 | // probability of infinite looping without making progress. |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 110 | MaxIterations += 2; |
Dan Gohman | d8db376 | 2009-07-15 16:35:29 +0000 | [diff] [blame] | 111 | errs() << "\n\n*** Shuffling does not hide the bug...\n\n"; |
Bill Wendling | 8ff4320 | 2008-02-26 10:46:10 +0000 | [diff] [blame] | 112 | } else { |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 113 | ShufflingEnabled = false; // Disable shuffling further on |
Dan Gohman | d8db376 | 2009-07-15 16:35:29 +0000 | [diff] [blame] | 114 | errs() << "\n\n*** Shuffling hides the bug...\n\n"; |
Bill Wendling | 8ff4320 | 2008-02-26 10:46:10 +0000 | [diff] [blame] | 115 | } |
| 116 | NumOfIterationsWithoutProgress = 0; |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 117 | } |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 118 | |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 119 | unsigned Mid = MidTop / 2; |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 120 | std::vector<ElTy> Prefix(TheList.begin(), TheList.begin() + Mid); |
| 121 | std::vector<ElTy> Suffix(TheList.begin() + Mid, TheList.end()); |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 122 | |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 123 | Expected<TestResult> Result = doTest(Prefix, Suffix); |
| 124 | if (Error E = Result.takeError()) |
Bill Wendling | c55cf4a | 2020-02-10 07:06:45 -0800 | [diff] [blame] | 125 | return std::move(E); |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 126 | switch (*Result) { |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 127 | case KeepSuffix: |
| 128 | // The property still holds. We can just drop the prefix elements, and |
| 129 | // shorten the list to the "kept" elements. |
Chris Lattner | 0347cda | 2003-04-25 03:16:33 +0000 | [diff] [blame] | 130 | TheList.swap(Suffix); |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 131 | MidTop = TheList.size(); |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 132 | // Reset progress treshold and progress counter |
| 133 | MaxIterations = MaxIterationsWithoutProgress; |
| 134 | NumOfIterationsWithoutProgress = 0; |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 135 | break; |
| 136 | case KeepPrefix: |
| 137 | // The predicate still holds, shorten the list to the prefix elements. |
| 138 | TheList.swap(Prefix); |
| 139 | MidTop = TheList.size(); |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 140 | // Reset progress treshold and progress counter |
| 141 | MaxIterations = MaxIterationsWithoutProgress; |
| 142 | NumOfIterationsWithoutProgress = 0; |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 143 | break; |
| 144 | case NoFailure: |
| 145 | // Otherwise the property doesn't hold. Some of the elements we removed |
Misha Brukman | 7eb05a1 | 2003-08-18 14:43:39 +0000 | [diff] [blame] | 146 | // must be necessary to maintain the property. |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 147 | MidTop = Mid; |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 148 | NumOfIterationsWithoutProgress++; |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 149 | break; |
| 150 | } |
| 151 | } |
| 152 | |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 153 | // Probability of backjumping from the trimming loop back to the binary |
| 154 | // split reduction loop. |
| 155 | const int BackjumpProbability = 10; |
| 156 | |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 157 | // Okay, we trimmed as much off the top and the bottom of the list as we |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 158 | // could. If there is more than two elements in the list, try deleting |
Reid Spencer | 9d59a82 | 2006-01-08 22:40:10 +0000 | [diff] [blame] | 159 | // interior elements and testing that. |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 160 | // |
Chris Lattner | 7fd27e2 | 2004-11-18 19:42:50 +0000 | [diff] [blame] | 161 | if (TheList.size() > 2) { |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 162 | bool Changed = true; |
| 163 | std::vector<ElTy> EmptyList; |
Tobias Grosser | 56eab36 | 2015-07-26 15:18:45 +0000 | [diff] [blame] | 164 | unsigned TrimIterations = 0; |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 165 | while (Changed) { // Trimming loop. |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 166 | Changed = false; |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 167 | |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 168 | // If the binary split reduction loop made an unfortunate sequence of |
| 169 | // splits, the trimming loop might be left off with a huge number of |
| 170 | // remaining elements (large search space). Backjumping out of that |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 171 | // search space and attempting a different split can significantly |
Chris Lattner | 8adfe92 | 2006-10-10 21:42:25 +0000 | [diff] [blame] | 172 | // improve the convergence speed. |
| 173 | if (std::rand() % 100 < BackjumpProbability) |
| 174 | goto Backjump; |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 175 | |
| 176 | for (unsigned i = 1; i < TheList.size() - 1; ++i) { |
| 177 | // Check interior elts |
Chris Lattner | beb01fa | 2005-08-02 02:16:17 +0000 | [diff] [blame] | 178 | if (BugpointIsInterrupted) { |
Dan Gohman | d8db376 | 2009-07-15 16:35:29 +0000 | [diff] [blame] | 179 | errs() << "\n\n*** Reduction Interrupted, cleaning up...\n\n"; |
Chris Lattner | beb01fa | 2005-08-02 02:16:17 +0000 | [diff] [blame] | 180 | return true; |
| 181 | } |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 182 | |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 183 | std::vector<ElTy> TestList(TheList); |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 184 | TestList.erase(TestList.begin() + i); |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 185 | |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 186 | Expected<TestResult> Result = doTest(EmptyList, TestList); |
| 187 | if (Error E = Result.takeError()) |
Bill Wendling | c55cf4a | 2020-02-10 07:06:45 -0800 | [diff] [blame] | 188 | return std::move(E); |
Justin Bogner | 1c03915 | 2016-09-06 17:18:22 +0000 | [diff] [blame] | 189 | if (*Result == KeepSuffix) { |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 190 | // We can trim down the list! |
| 191 | TheList.swap(TestList); |
Justin Bogner | 06d466a | 2016-09-01 21:04:36 +0000 | [diff] [blame] | 192 | --i; // Don't skip an element of the list |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 193 | Changed = true; |
Justin Bogner | 46b1a9a | 2016-09-06 04:04:13 +0000 | [diff] [blame] | 194 | } |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 195 | } |
Tobias Grosser | 56eab36 | 2015-07-26 15:18:45 +0000 | [diff] [blame] | 196 | if (TrimIterations >= MaxTrimIterationsWithoutBackJump) |
| 197 | break; |
| 198 | TrimIterations++; |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 199 | } |
| 200 | } |
Misha Brukman | 87154fd | 2003-07-30 20:15:56 +0000 | [diff] [blame] | 201 | |
| 202 | return true; // there are some failure and we've narrowed them down |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 203 | } |
| 204 | }; |
| 205 | |
Brian Gaeke | 960707c | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 206 | } // End llvm namespace |
| 207 | |
Chris Lattner | 9f5a197 | 2003-04-24 20:16:29 +0000 | [diff] [blame] | 208 | #endif |