Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 1 | //===--- TextDiagnosticPrinter.cpp - Diagnostic Printer -------------------===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | 5b12ab8 | 2007-12-29 19:59:25 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // This diagnostic client prints out their diagnostic messages. |
| 11 | // |
| 12 | //===----------------------------------------------------------------------===// |
| 13 | |
Daniel Dunbar | 51adf58 | 2009-03-02 06:16:29 +0000 | [diff] [blame] | 14 | #include "clang/Frontend/TextDiagnosticPrinter.h" |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 15 | #include "clang/Basic/SourceManager.h" |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 16 | #include "clang/Frontend/DiagnosticOptions.h" |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 17 | #include "clang/Lex/Lexer.h" |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 18 | #include "llvm/Support/MemoryBuffer.h" |
Chris Lattner | 327984f | 2008-11-19 06:56:25 +0000 | [diff] [blame] | 19 | #include "llvm/Support/raw_ostream.h" |
Chris Lattner | 23be067 | 2008-11-19 06:51:40 +0000 | [diff] [blame] | 20 | #include "llvm/ADT/SmallString.h" |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 21 | #include <algorithm> |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 22 | using namespace clang; |
| 23 | |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 24 | static const enum llvm::raw_ostream::Colors noteColor = |
| 25 | llvm::raw_ostream::BLACK; |
| 26 | static const enum llvm::raw_ostream::Colors fixitColor = |
| 27 | llvm::raw_ostream::GREEN; |
| 28 | static const enum llvm::raw_ostream::Colors caretColor = |
| 29 | llvm::raw_ostream::GREEN; |
| 30 | static const enum llvm::raw_ostream::Colors warningColor = |
| 31 | llvm::raw_ostream::MAGENTA; |
| 32 | static const enum llvm::raw_ostream::Colors errorColor = llvm::raw_ostream::RED; |
| 33 | static const enum llvm::raw_ostream::Colors fatalColor = llvm::raw_ostream::RED; |
| 34 | // used for changing only the bold attribute |
| 35 | static const enum llvm::raw_ostream::Colors savedColor = |
| 36 | llvm::raw_ostream::SAVEDCOLOR; |
| 37 | |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 38 | /// \brief Number of spaces to indent when word-wrapping. |
| 39 | const unsigned WordWrapIndentation = 6; |
| 40 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 41 | TextDiagnosticPrinter::TextDiagnosticPrinter(llvm::raw_ostream &os, |
| 42 | const DiagnosticOptions &diags) |
| 43 | : OS(os), LangOpts(0), DiagOpts(&diags), |
| 44 | LastCaretDiagnosticWasNote(false) { |
| 45 | } |
| 46 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 47 | void TextDiagnosticPrinter:: |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 48 | PrintIncludeStack(SourceLocation Loc, const SourceManager &SM) { |
| 49 | if (Loc.isInvalid()) return; |
Chris Lattner | dc5c055 | 2007-07-20 16:37:10 +0000 | [diff] [blame] | 50 | |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 51 | PresumedLoc PLoc = SM.getPresumedLoc(Loc); |
Chris Lattner | dc5c055 | 2007-07-20 16:37:10 +0000 | [diff] [blame] | 52 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 53 | // Print out the other include frames first. |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 54 | PrintIncludeStack(PLoc.getIncludeLoc(), SM); |
Chris Lattner | 92b29b2f | 2009-04-21 03:57:54 +0000 | [diff] [blame] | 55 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 56 | if (DiagOpts->ShowLocation) |
Chris Lattner | 92b29b2f | 2009-04-21 03:57:54 +0000 | [diff] [blame] | 57 | OS << "In file included from " << PLoc.getFilename() |
| 58 | << ':' << PLoc.getLine() << ":\n"; |
| 59 | else |
| 60 | OS << "In included file:\n"; |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 61 | } |
| 62 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 63 | /// HighlightRange - Given a SourceRange and a line number, highlight (with ~'s) |
| 64 | /// any characters in LineNo that intersect the SourceRange. |
Ted Kremenek | d4e5fba | 2007-12-11 21:27:55 +0000 | [diff] [blame] | 65 | void TextDiagnosticPrinter::HighlightRange(const SourceRange &R, |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 66 | const SourceManager &SM, |
Chris Lattner | 71dc14b | 2009-01-17 08:45:21 +0000 | [diff] [blame] | 67 | unsigned LineNo, FileID FID, |
Gordon Henriksen | 04c50bd | 2008-08-09 19:58:22 +0000 | [diff] [blame] | 68 | std::string &CaretLine, |
Nuno Lopes | d86aa93 | 2008-08-05 19:40:20 +0000 | [diff] [blame] | 69 | const std::string &SourceLine) { |
Gordon Henriksen | 04c50bd | 2008-08-09 19:58:22 +0000 | [diff] [blame] | 70 | assert(CaretLine.size() == SourceLine.size() && |
| 71 | "Expect a correspondence between source and caret line!"); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 72 | if (!R.isValid()) return; |
| 73 | |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 74 | SourceLocation Begin = SM.getInstantiationLoc(R.getBegin()); |
| 75 | SourceLocation End = SM.getInstantiationLoc(R.getEnd()); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 76 | |
Chris Lattner | b42700f | 2009-02-17 05:19:10 +0000 | [diff] [blame] | 77 | // If the End location and the start location are the same and are a macro |
| 78 | // location, then the range was something that came from a macro expansion |
| 79 | // or _Pragma. If this is an object-like macro, the best we can do is to |
| 80 | // highlight the range. If this is a function-like macro, we'd also like to |
| 81 | // highlight the arguments. |
| 82 | if (Begin == End && R.getEnd().isMacroID()) |
| 83 | End = SM.getInstantiationRange(R.getEnd()).second; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 84 | |
Chris Lattner | 88ea93e | 2009-02-04 01:06:56 +0000 | [diff] [blame] | 85 | unsigned StartLineNo = SM.getInstantiationLineNumber(Begin); |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 86 | if (StartLineNo > LineNo || SM.getFileID(Begin) != FID) |
Chris Lattner | 177977ac | 2008-01-12 06:43:35 +0000 | [diff] [blame] | 87 | return; // No intersection. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 88 | |
Chris Lattner | 88ea93e | 2009-02-04 01:06:56 +0000 | [diff] [blame] | 89 | unsigned EndLineNo = SM.getInstantiationLineNumber(End); |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 90 | if (EndLineNo < LineNo || SM.getFileID(End) != FID) |
Chris Lattner | 177977ac | 2008-01-12 06:43:35 +0000 | [diff] [blame] | 91 | return; // No intersection. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 92 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 93 | // Compute the column number of the start. |
| 94 | unsigned StartColNo = 0; |
| 95 | if (StartLineNo == LineNo) { |
Chris Lattner | e4ad417 | 2009-02-04 00:55:58 +0000 | [diff] [blame] | 96 | StartColNo = SM.getInstantiationColumnNumber(Begin); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 97 | if (StartColNo) --StartColNo; // Zero base the col #. |
| 98 | } |
| 99 | |
| 100 | // Pick the first non-whitespace column. |
| 101 | while (StartColNo < SourceLine.size() && |
| 102 | (SourceLine[StartColNo] == ' ' || SourceLine[StartColNo] == '\t')) |
| 103 | ++StartColNo; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 104 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 105 | // Compute the column number of the end. |
Gordon Henriksen | 04c50bd | 2008-08-09 19:58:22 +0000 | [diff] [blame] | 106 | unsigned EndColNo = CaretLine.size(); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 107 | if (EndLineNo == LineNo) { |
Chris Lattner | e4ad417 | 2009-02-04 00:55:58 +0000 | [diff] [blame] | 108 | EndColNo = SM.getInstantiationColumnNumber(End); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 109 | if (EndColNo) { |
| 110 | --EndColNo; // Zero base the col #. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 111 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 112 | // Add in the length of the token, so that we cover multi-char tokens. |
Chris Lattner | 184e65d | 2009-04-14 23:22:57 +0000 | [diff] [blame] | 113 | EndColNo += Lexer::MeasureTokenLength(End, SM, *LangOpts); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 114 | } else { |
Gordon Henriksen | 04c50bd | 2008-08-09 19:58:22 +0000 | [diff] [blame] | 115 | EndColNo = CaretLine.size(); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 116 | } |
| 117 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 118 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 119 | // Pick the last non-whitespace column. |
Nuno Lopes | d86aa93 | 2008-08-05 19:40:20 +0000 | [diff] [blame] | 120 | if (EndColNo <= SourceLine.size()) |
| 121 | while (EndColNo-1 && |
| 122 | (SourceLine[EndColNo-1] == ' ' || SourceLine[EndColNo-1] == '\t')) |
| 123 | --EndColNo; |
| 124 | else |
| 125 | EndColNo = SourceLine.size(); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 126 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 127 | // Fill the range with ~'s. |
| 128 | assert(StartColNo <= EndColNo && "Invalid range!"); |
Nuno Lopes | d86aa93 | 2008-08-05 19:40:20 +0000 | [diff] [blame] | 129 | for (unsigned i = StartColNo; i < EndColNo; ++i) |
Gordon Henriksen | 04c50bd | 2008-08-09 19:58:22 +0000 | [diff] [blame] | 130 | CaretLine[i] = '~'; |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 131 | } |
| 132 | |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 133 | /// \brief When the source code line we want to print is too long for |
| 134 | /// the terminal, select the "interesting" region. |
| 135 | static void SelectInterestingSourceRegion(std::string &SourceLine, |
| 136 | std::string &CaretLine, |
| 137 | std::string &FixItInsertionLine, |
Douglas Gregor | 2d69cd2 | 2009-05-04 06:27:32 +0000 | [diff] [blame] | 138 | unsigned EndOfCaretToken, |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 139 | unsigned Columns) { |
| 140 | if (CaretLine.size() > SourceLine.size()) |
| 141 | SourceLine.resize(CaretLine.size(), ' '); |
| 142 | |
| 143 | // Find the slice that we need to display the full caret line |
| 144 | // correctly. |
| 145 | unsigned CaretStart = 0, CaretEnd = CaretLine.size(); |
| 146 | for (; CaretStart != CaretEnd; ++CaretStart) |
| 147 | if (!isspace(CaretLine[CaretStart])) |
| 148 | break; |
| 149 | |
| 150 | for (; CaretEnd != CaretStart; --CaretEnd) |
| 151 | if (!isspace(CaretLine[CaretEnd - 1])) |
| 152 | break; |
Douglas Gregor | 2d69cd2 | 2009-05-04 06:27:32 +0000 | [diff] [blame] | 153 | |
| 154 | // Make sure we don't chop the string shorter than the caret token |
| 155 | // itself. |
| 156 | if (CaretEnd < EndOfCaretToken) |
| 157 | CaretEnd = EndOfCaretToken; |
| 158 | |
Douglas Gregor | b8c6d5d | 2009-05-03 04:33:32 +0000 | [diff] [blame] | 159 | // If we have a fix-it line, make sure the slice includes all of the |
| 160 | // fix-it information. |
| 161 | if (!FixItInsertionLine.empty()) { |
| 162 | unsigned FixItStart = 0, FixItEnd = FixItInsertionLine.size(); |
| 163 | for (; FixItStart != FixItEnd; ++FixItStart) |
| 164 | if (!isspace(FixItInsertionLine[FixItStart])) |
| 165 | break; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 166 | |
Douglas Gregor | b8c6d5d | 2009-05-03 04:33:32 +0000 | [diff] [blame] | 167 | for (; FixItEnd != FixItStart; --FixItEnd) |
| 168 | if (!isspace(FixItInsertionLine[FixItEnd - 1])) |
| 169 | break; |
| 170 | |
| 171 | if (FixItStart < CaretStart) |
| 172 | CaretStart = FixItStart; |
| 173 | if (FixItEnd > CaretEnd) |
| 174 | CaretEnd = FixItEnd; |
| 175 | } |
| 176 | |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 177 | // CaretLine[CaretStart, CaretEnd) contains all of the interesting |
| 178 | // parts of the caret line. While this slice is smaller than the |
| 179 | // number of columns we have, try to grow the slice to encompass |
| 180 | // more context. |
| 181 | |
| 182 | // If the end of the interesting region comes before we run out of |
| 183 | // space in the terminal, start at the beginning of the line. |
Douglas Gregor | 12c3a5c | 2009-05-15 18:05:24 +0000 | [diff] [blame] | 184 | if (Columns > 3 && CaretEnd < Columns - 3) |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 185 | CaretStart = 0; |
| 186 | |
Douglas Gregor | 12c3a5c | 2009-05-15 18:05:24 +0000 | [diff] [blame] | 187 | unsigned TargetColumns = Columns; |
| 188 | if (TargetColumns > 8) |
| 189 | TargetColumns -= 8; // Give us extra room for the ellipses. |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 190 | unsigned SourceLength = SourceLine.size(); |
Douglas Gregor | 006fd38 | 2009-05-04 06:45:38 +0000 | [diff] [blame] | 191 | while ((CaretEnd - CaretStart) < TargetColumns) { |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 192 | bool ExpandedRegion = false; |
| 193 | // Move the start of the interesting region left until we've |
| 194 | // pulled in something else interesting. |
Douglas Gregor | 006fd38 | 2009-05-04 06:45:38 +0000 | [diff] [blame] | 195 | if (CaretStart == 1) |
| 196 | CaretStart = 0; |
| 197 | else if (CaretStart > 1) { |
| 198 | unsigned NewStart = CaretStart - 1; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 199 | |
Douglas Gregor | 006fd38 | 2009-05-04 06:45:38 +0000 | [diff] [blame] | 200 | // Skip over any whitespace we see here; we're looking for |
| 201 | // another bit of interesting text. |
| 202 | while (NewStart && isspace(SourceLine[NewStart])) |
| 203 | --NewStart; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 204 | |
Douglas Gregor | 006fd38 | 2009-05-04 06:45:38 +0000 | [diff] [blame] | 205 | // Skip over this bit of "interesting" text. |
| 206 | while (NewStart && !isspace(SourceLine[NewStart])) |
| 207 | --NewStart; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 208 | |
Douglas Gregor | 006fd38 | 2009-05-04 06:45:38 +0000 | [diff] [blame] | 209 | // Move up to the non-whitespace character we just saw. |
| 210 | if (NewStart) |
| 211 | ++NewStart; |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 212 | |
| 213 | // If we're still within our limit, update the starting |
| 214 | // position within the source/caret line. |
Douglas Gregor | 006fd38 | 2009-05-04 06:45:38 +0000 | [diff] [blame] | 215 | if (CaretEnd - NewStart <= TargetColumns) { |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 216 | CaretStart = NewStart; |
| 217 | ExpandedRegion = true; |
| 218 | } |
| 219 | } |
| 220 | |
| 221 | // Move the end of the interesting region right until we've |
| 222 | // pulled in something else interesting. |
Daniel Dunbar | 6f94991 | 2009-05-03 23:04:40 +0000 | [diff] [blame] | 223 | if (CaretEnd != SourceLength) { |
Daniel Dunbar | 28a24fd | 2009-10-19 09:11:21 +0000 | [diff] [blame] | 224 | assert(CaretEnd < SourceLength && "Unexpected caret position!"); |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 225 | unsigned NewEnd = CaretEnd; |
| 226 | |
| 227 | // Skip over any whitespace we see here; we're looking for |
| 228 | // another bit of interesting text. |
Douglas Gregor | 8e35e2d | 2009-05-18 22:09:16 +0000 | [diff] [blame] | 229 | while (NewEnd != SourceLength && isspace(SourceLine[NewEnd - 1])) |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 230 | ++NewEnd; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 231 | |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 232 | // Skip over this bit of "interesting" text. |
Douglas Gregor | 8e35e2d | 2009-05-18 22:09:16 +0000 | [diff] [blame] | 233 | while (NewEnd != SourceLength && !isspace(SourceLine[NewEnd - 1])) |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 234 | ++NewEnd; |
| 235 | |
| 236 | if (NewEnd - CaretStart <= TargetColumns) { |
| 237 | CaretEnd = NewEnd; |
| 238 | ExpandedRegion = true; |
| 239 | } |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 240 | } |
Daniel Dunbar | 6f94991 | 2009-05-03 23:04:40 +0000 | [diff] [blame] | 241 | |
| 242 | if (!ExpandedRegion) |
| 243 | break; |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 244 | } |
| 245 | |
| 246 | // [CaretStart, CaretEnd) is the slice we want. Update the various |
| 247 | // output lines to show only this slice, with two-space padding |
| 248 | // before the lines so that it looks nicer. |
Douglas Gregor | 54c6a3b | 2009-05-03 04:12:51 +0000 | [diff] [blame] | 249 | if (CaretEnd < SourceLine.size()) |
| 250 | SourceLine.replace(CaretEnd, std::string::npos, "..."); |
Douglas Gregor | cb51662 | 2009-05-03 15:24:25 +0000 | [diff] [blame] | 251 | if (CaretEnd < CaretLine.size()) |
| 252 | CaretLine.erase(CaretEnd, std::string::npos); |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 253 | if (FixItInsertionLine.size() > CaretEnd) |
| 254 | FixItInsertionLine.erase(CaretEnd, std::string::npos); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 255 | |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 256 | if (CaretStart > 2) { |
Douglas Gregor | 54c6a3b | 2009-05-03 04:12:51 +0000 | [diff] [blame] | 257 | SourceLine.replace(0, CaretStart, " ..."); |
| 258 | CaretLine.replace(0, CaretStart, " "); |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 259 | if (FixItInsertionLine.size() >= CaretStart) |
Douglas Gregor | 54c6a3b | 2009-05-03 04:12:51 +0000 | [diff] [blame] | 260 | FixItInsertionLine.replace(0, CaretStart, " "); |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 261 | } |
| 262 | } |
| 263 | |
Chris Lattner | 1973d84 | 2009-02-20 00:18:51 +0000 | [diff] [blame] | 264 | void TextDiagnosticPrinter::EmitCaretDiagnostic(SourceLocation Loc, |
Chris Lattner | 464ceb4 | 2009-02-20 00:25:28 +0000 | [diff] [blame] | 265 | SourceRange *Ranges, |
Chris Lattner | 1973d84 | 2009-02-20 00:18:51 +0000 | [diff] [blame] | 266 | unsigned NumRanges, |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 267 | SourceManager &SM, |
| 268 | const CodeModificationHint *Hints, |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 269 | unsigned NumHints, |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 270 | unsigned Columns) { |
Daniel Dunbar | 0231895 | 2009-11-05 02:42:12 +0000 | [diff] [blame^] | 271 | assert(LangOpts && "Unexpected diagnostic outside source file processing"); |
Chris Lattner | 7b60a16 | 2009-02-17 08:44:50 +0000 | [diff] [blame] | 272 | assert(!Loc.isInvalid() && "must have a valid source location here"); |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 273 | |
| 274 | // If this is a macro ID, first emit information about where this was |
| 275 | // instantiated (recursively) then emit information about where. the token was |
| 276 | // spelled from. |
Chris Lattner | 7b60a16 | 2009-02-17 08:44:50 +0000 | [diff] [blame] | 277 | if (!Loc.isFileID()) { |
Chris Lattner | 24eb28b | 2009-02-18 18:50:45 +0000 | [diff] [blame] | 278 | SourceLocation OneLevelUp = SM.getImmediateInstantiationRange(Loc).first; |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 279 | // FIXME: Map ranges? |
Douglas Gregor | 080fe61 | 2009-05-06 04:43:47 +0000 | [diff] [blame] | 280 | EmitCaretDiagnostic(OneLevelUp, Ranges, NumRanges, SM, 0, 0, Columns); |
Chris Lattner | 464ceb4 | 2009-02-20 00:25:28 +0000 | [diff] [blame] | 281 | |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 282 | Loc = SM.getImmediateSpellingLoc(Loc); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 283 | |
Chris Lattner | 464ceb4 | 2009-02-20 00:25:28 +0000 | [diff] [blame] | 284 | // Map the ranges. |
| 285 | for (unsigned i = 0; i != NumRanges; ++i) { |
| 286 | SourceLocation S = Ranges[i].getBegin(), E = Ranges[i].getEnd(); |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 287 | if (S.isMacroID()) S = SM.getImmediateSpellingLoc(S); |
| 288 | if (E.isMacroID()) E = SM.getImmediateSpellingLoc(E); |
Chris Lattner | 464ceb4 | 2009-02-20 00:25:28 +0000 | [diff] [blame] | 289 | Ranges[i] = SourceRange(S, E); |
| 290 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 291 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 292 | if (DiagOpts->ShowLocation) { |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 293 | std::pair<FileID, unsigned> IInfo = SM.getDecomposedInstantiationLoc(Loc); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 294 | |
Chris Lattner | 92b29b2f | 2009-04-21 03:57:54 +0000 | [diff] [blame] | 295 | // Emit the file/line/column that this expansion came from. |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 296 | OS << SM.getBuffer(IInfo.first)->getBufferIdentifier() << ':' |
| 297 | << SM.getLineNumber(IInfo.first, IInfo.second) << ':'; |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 298 | if (DiagOpts->ShowColumn) |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 299 | OS << SM.getColumnNumber(IInfo.first, IInfo.second) << ':'; |
Chris Lattner | 92b29b2f | 2009-04-21 03:57:54 +0000 | [diff] [blame] | 300 | OS << ' '; |
| 301 | } |
| 302 | OS << "note: instantiated from:\n"; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 303 | |
Douglas Gregor | 080fe61 | 2009-05-06 04:43:47 +0000 | [diff] [blame] | 304 | EmitCaretDiagnostic(Loc, Ranges, NumRanges, SM, Hints, NumHints, Columns); |
Chris Lattner | c45529b | 2009-05-05 22:03:18 +0000 | [diff] [blame] | 305 | return; |
Chris Lattner | 7b60a16 | 2009-02-17 08:44:50 +0000 | [diff] [blame] | 306 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 307 | |
Chris Lattner | d22603c | 2009-02-17 07:51:53 +0000 | [diff] [blame] | 308 | // Decompose the location into a FID/Offset pair. |
| 309 | std::pair<FileID, unsigned> LocInfo = SM.getDecomposedLoc(Loc); |
| 310 | FileID FID = LocInfo.first; |
| 311 | unsigned FileOffset = LocInfo.second; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 312 | |
Chris Lattner | d22603c | 2009-02-17 07:51:53 +0000 | [diff] [blame] | 313 | // Get information about the buffer it points into. |
| 314 | std::pair<const char*, const char*> BufferInfo = SM.getBufferData(FID); |
| 315 | const char *BufStart = BufferInfo.first; |
Chris Lattner | d22603c | 2009-02-17 07:51:53 +0000 | [diff] [blame] | 316 | |
| 317 | unsigned ColNo = SM.getColumnNumber(FID, FileOffset); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 318 | unsigned CaretEndColNo |
Douglas Gregor | 2d69cd2 | 2009-05-04 06:27:32 +0000 | [diff] [blame] | 319 | = ColNo + Lexer::MeasureTokenLength(Loc, SM, *LangOpts); |
| 320 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 321 | // Rewind from the current position to the start of the line. |
Chris Lattner | d22603c | 2009-02-17 07:51:53 +0000 | [diff] [blame] | 322 | const char *TokPtr = BufStart+FileOffset; |
| 323 | const char *LineStart = TokPtr-ColNo+1; // Column # is 1-based. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 324 | |
| 325 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 326 | // Compute the line end. Scan forward from the error position to the end of |
| 327 | // the line. |
Chris Lattner | d22603c | 2009-02-17 07:51:53 +0000 | [diff] [blame] | 328 | const char *LineEnd = TokPtr; |
Chris Lattner | 139d87b | 2009-03-08 08:11:22 +0000 | [diff] [blame] | 329 | while (*LineEnd != '\n' && *LineEnd != '\r' && *LineEnd != '\0') |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 330 | ++LineEnd; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 331 | |
Daniel Dunbar | 28a24fd | 2009-10-19 09:11:21 +0000 | [diff] [blame] | 332 | // FIXME: This shouldn't be necessary, but the CaretEndColNo can extend past |
| 333 | // the source line length as currently being computed. See |
| 334 | // test/Misc/message-length.c. |
| 335 | CaretEndColNo = std::min(CaretEndColNo, unsigned(LineEnd - LineStart)); |
| 336 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 337 | // Copy the line of code into an std::string for ease of manipulation. |
| 338 | std::string SourceLine(LineStart, LineEnd); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 339 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 340 | // Create a line for the caret that is filled with spaces that is the same |
| 341 | // length as the line of source code. |
| 342 | std::string CaretLine(LineEnd-LineStart, ' '); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 343 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 344 | // Highlight all of the characters covered by Ranges with ~ characters. |
Chris Lattner | 1973d84 | 2009-02-20 00:18:51 +0000 | [diff] [blame] | 345 | if (NumRanges) { |
Chris Lattner | d22603c | 2009-02-17 07:51:53 +0000 | [diff] [blame] | 346 | unsigned LineNo = SM.getLineNumber(FID, FileOffset); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 347 | |
Chris Lattner | 1973d84 | 2009-02-20 00:18:51 +0000 | [diff] [blame] | 348 | for (unsigned i = 0, e = NumRanges; i != e; ++i) |
| 349 | HighlightRange(Ranges[i], SM, LineNo, FID, CaretLine, SourceLine); |
Chris Lattner | d22603c | 2009-02-17 07:51:53 +0000 | [diff] [blame] | 350 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 351 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 352 | // Next, insert the caret itself. |
| 353 | if (ColNo-1 < CaretLine.size()) |
| 354 | CaretLine[ColNo-1] = '^'; |
| 355 | else |
| 356 | CaretLine.push_back('^'); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 357 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 358 | // Scan the source line, looking for tabs. If we find any, manually expand |
| 359 | // them to 8 characters and update the CaretLine to match. |
| 360 | for (unsigned i = 0; i != SourceLine.size(); ++i) { |
| 361 | if (SourceLine[i] != '\t') continue; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 362 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 363 | // Replace this tab with at least one space. |
| 364 | SourceLine[i] = ' '; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 365 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 366 | // Compute the number of spaces we need to insert. |
| 367 | unsigned NumSpaces = ((i+8)&~7) - (i+1); |
| 368 | assert(NumSpaces < 8 && "Invalid computation of space amt"); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 369 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 370 | // Insert spaces into the SourceLine. |
| 371 | SourceLine.insert(i+1, NumSpaces, ' '); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 372 | |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 373 | // Insert spaces or ~'s into CaretLine. |
| 374 | CaretLine.insert(i+1, NumSpaces, CaretLine[i] == '~' ? '~' : ' '); |
| 375 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 376 | |
Chris Lattner | af6094e | 2009-04-28 22:33:16 +0000 | [diff] [blame] | 377 | // If we are in -fdiagnostics-print-source-range-info mode, we are trying to |
| 378 | // produce easily machine parsable output. Add a space before the source line |
| 379 | // and the caret to make it trivial to tell the main diagnostic line from what |
| 380 | // the user is intended to see. |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 381 | if (DiagOpts->ShowSourceRanges) { |
Chris Lattner | af6094e | 2009-04-28 22:33:16 +0000 | [diff] [blame] | 382 | SourceLine = ' ' + SourceLine; |
| 383 | CaretLine = ' ' + CaretLine; |
| 384 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 385 | |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 386 | std::string FixItInsertionLine; |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 387 | if (NumHints && DiagOpts->ShowFixits) { |
Chris Lattner | 29d34ca | 2009-04-19 07:44:08 +0000 | [diff] [blame] | 388 | for (const CodeModificationHint *Hint = Hints, *LastHint = Hints + NumHints; |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 389 | Hint != LastHint; ++Hint) { |
| 390 | if (Hint->InsertionLoc.isValid()) { |
| 391 | // We have an insertion hint. Determine whether the inserted |
| 392 | // code is on the same line as the caret. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 393 | std::pair<FileID, unsigned> HintLocInfo |
Chris Lattner | e29509a | 2009-03-02 20:58:48 +0000 | [diff] [blame] | 394 | = SM.getDecomposedInstantiationLoc(Hint->InsertionLoc); |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 395 | if (SM.getLineNumber(HintLocInfo.first, HintLocInfo.second) == |
| 396 | SM.getLineNumber(FID, FileOffset)) { |
| 397 | // Insert the new code into the line just below the code |
| 398 | // that the user wrote. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 399 | unsigned HintColNo |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 400 | = SM.getColumnNumber(HintLocInfo.first, HintLocInfo.second); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 401 | unsigned LastColumnModified |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 402 | = HintColNo - 1 + Hint->CodeToInsert.size(); |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 403 | if (LastColumnModified > FixItInsertionLine.size()) |
| 404 | FixItInsertionLine.resize(LastColumnModified, ' '); |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 405 | std::copy(Hint->CodeToInsert.begin(), Hint->CodeToInsert.end(), |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 406 | FixItInsertionLine.begin() + HintColNo - 1); |
Douglas Gregor | b8c6d5d | 2009-05-03 04:33:32 +0000 | [diff] [blame] | 407 | } else { |
| 408 | FixItInsertionLine.clear(); |
| 409 | break; |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 410 | } |
| 411 | } |
| 412 | } |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 413 | } |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 414 | |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 415 | // If the source line is too long for our terminal, select only the |
| 416 | // "interesting" source region within that line. |
| 417 | if (Columns && SourceLine.size() > Columns) |
| 418 | SelectInterestingSourceRegion(SourceLine, CaretLine, FixItInsertionLine, |
Douglas Gregor | 2d69cd2 | 2009-05-04 06:27:32 +0000 | [diff] [blame] | 419 | CaretEndColNo, Columns); |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 420 | |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 421 | // Finally, remove any blank spaces from the end of CaretLine. |
| 422 | while (CaretLine[CaretLine.size()-1] == ' ') |
| 423 | CaretLine.erase(CaretLine.end()-1); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 424 | |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 425 | // Emit what we have computed. |
| 426 | OS << SourceLine << '\n'; |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 427 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 428 | if (DiagOpts->ShowColors) |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 429 | OS.changeColor(caretColor, true); |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 430 | OS << CaretLine << '\n'; |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 431 | if (DiagOpts->ShowColors) |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 432 | OS.resetColor(); |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 433 | |
| 434 | if (!FixItInsertionLine.empty()) { |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 435 | if (DiagOpts->ShowColors) |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 436 | // Print fixit line in color |
| 437 | OS.changeColor(fixitColor, false); |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 438 | if (DiagOpts->ShowSourceRanges) |
Douglas Gregor | cf7b2af | 2009-05-01 23:32:58 +0000 | [diff] [blame] | 439 | OS << ' '; |
| 440 | OS << FixItInsertionLine << '\n'; |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 441 | if (DiagOpts->ShowColors) |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 442 | OS.resetColor(); |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 443 | } |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 444 | } |
| 445 | |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 446 | /// \brief Skip over whitespace in the string, starting at the given |
| 447 | /// index. |
| 448 | /// |
| 449 | /// \returns The index of the first non-whitespace character that is |
| 450 | /// greater than or equal to Idx or, if no such character exists, |
| 451 | /// returns the end of the string. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 452 | static unsigned skipWhitespace(unsigned Idx, |
Mike Stump | 11289f4 | 2009-09-09 15:08:12 +0000 | [diff] [blame] | 453 | const llvm::SmallVectorImpl<char> &Str, |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 454 | unsigned Length) { |
| 455 | while (Idx < Length && isspace(Str[Idx])) |
| 456 | ++Idx; |
| 457 | return Idx; |
| 458 | } |
| 459 | |
| 460 | /// \brief If the given character is the start of some kind of |
| 461 | /// balanced punctuation (e.g., quotes or parentheses), return the |
| 462 | /// character that will terminate the punctuation. |
| 463 | /// |
| 464 | /// \returns The ending punctuation character, if any, or the NULL |
| 465 | /// character if the input character does not start any punctuation. |
| 466 | static inline char findMatchingPunctuation(char c) { |
| 467 | switch (c) { |
| 468 | case '\'': return '\''; |
| 469 | case '`': return '\''; |
| 470 | case '"': return '"'; |
| 471 | case '(': return ')'; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 472 | case '[': return ']'; |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 473 | case '{': return '}'; |
| 474 | default: break; |
| 475 | } |
| 476 | |
| 477 | return 0; |
| 478 | } |
| 479 | |
| 480 | /// \brief Find the end of the word starting at the given offset |
| 481 | /// within a string. |
| 482 | /// |
| 483 | /// \returns the index pointing one character past the end of the |
| 484 | /// word. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 485 | unsigned findEndOfWord(unsigned Start, |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 486 | const llvm::SmallVectorImpl<char> &Str, |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 487 | unsigned Length, unsigned Column, |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 488 | unsigned Columns) { |
| 489 | unsigned End = Start + 1; |
| 490 | |
| 491 | // Determine if the start of the string is actually opening |
| 492 | // punctuation, e.g., a quote or parentheses. |
| 493 | char EndPunct = findMatchingPunctuation(Str[Start]); |
| 494 | if (!EndPunct) { |
| 495 | // This is a normal word. Just find the first space character. |
| 496 | while (End < Length && !isspace(Str[End])) |
| 497 | ++End; |
| 498 | return End; |
| 499 | } |
| 500 | |
| 501 | // We have the start of a balanced punctuation sequence (quotes, |
| 502 | // parentheses, etc.). Determine the full sequence is. |
| 503 | llvm::SmallVector<char, 16> PunctuationEndStack; |
| 504 | PunctuationEndStack.push_back(EndPunct); |
| 505 | while (End < Length && !PunctuationEndStack.empty()) { |
| 506 | if (Str[End] == PunctuationEndStack.back()) |
| 507 | PunctuationEndStack.pop_back(); |
| 508 | else if (char SubEndPunct = findMatchingPunctuation(Str[End])) |
| 509 | PunctuationEndStack.push_back(SubEndPunct); |
| 510 | |
| 511 | ++End; |
| 512 | } |
| 513 | |
| 514 | // Find the first space character after the punctuation ended. |
| 515 | while (End < Length && !isspace(Str[End])) |
| 516 | ++End; |
| 517 | |
| 518 | unsigned PunctWordLength = End - Start; |
| 519 | if (// If the word fits on this line |
| 520 | Column + PunctWordLength <= Columns || |
| 521 | // ... or the word is "short enough" to take up the next line |
| 522 | // without too much ugly white space |
| 523 | PunctWordLength < Columns/3) |
| 524 | return End; // Take the whole thing as a single "word". |
| 525 | |
| 526 | // The whole quoted/parenthesized string is too long to print as a |
| 527 | // single "word". Instead, find the "word" that starts just after |
| 528 | // the punctuation and use that end-point instead. This will recurse |
| 529 | // until it finds something small enough to consider a word. |
| 530 | return findEndOfWord(Start + 1, Str, Length, Column + 1, Columns); |
| 531 | } |
| 532 | |
| 533 | /// \brief Print the given string to a stream, word-wrapping it to |
| 534 | /// some number of columns in the process. |
| 535 | /// |
| 536 | /// \brief OS the stream to which the word-wrapping string will be |
| 537 | /// emitted. |
| 538 | /// |
| 539 | /// \brief Str the string to word-wrap and output. |
| 540 | /// |
| 541 | /// \brief Columns the number of columns to word-wrap to. |
| 542 | /// |
| 543 | /// \brief Column the column number at which the first character of \p |
| 544 | /// Str will be printed. This will be non-zero when part of the first |
| 545 | /// line has already been printed. |
| 546 | /// |
| 547 | /// \brief Indentation the number of spaces to indent any lines beyond |
| 548 | /// the first line. |
| 549 | /// |
| 550 | /// \returns true if word-wrapping was required, or false if the |
| 551 | /// string fit on the first line. |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 552 | static bool PrintWordWrapped(llvm::raw_ostream &OS, |
Mike Stump | 11289f4 | 2009-09-09 15:08:12 +0000 | [diff] [blame] | 553 | const llvm::SmallVectorImpl<char> &Str, |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 554 | unsigned Columns, |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 555 | unsigned Column = 0, |
| 556 | unsigned Indentation = WordWrapIndentation) { |
| 557 | unsigned Length = Str.size(); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 558 | |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 559 | // If there is a newline in this message somewhere, find that |
| 560 | // newline and split the message into the part before the newline |
| 561 | // (which will be word-wrapped) and the part from the newline one |
| 562 | // (which will be emitted unchanged). |
| 563 | for (unsigned I = 0; I != Length; ++I) |
| 564 | if (Str[I] == '\n') { |
| 565 | Length = I; |
| 566 | break; |
| 567 | } |
| 568 | |
| 569 | // The string used to indent each line. |
| 570 | llvm::SmallString<16> IndentStr; |
| 571 | IndentStr.assign(Indentation, ' '); |
| 572 | bool Wrapped = false; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 573 | for (unsigned WordStart = 0, WordEnd; WordStart < Length; |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 574 | WordStart = WordEnd) { |
| 575 | // Find the beginning of the next word. |
| 576 | WordStart = skipWhitespace(WordStart, Str, Length); |
| 577 | if (WordStart == Length) |
| 578 | break; |
| 579 | |
| 580 | // Find the end of this word. |
| 581 | WordEnd = findEndOfWord(WordStart, Str, Length, Column, Columns); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 582 | |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 583 | // Does this word fit on the current line? |
| 584 | unsigned WordLength = WordEnd - WordStart; |
| 585 | if (Column + WordLength < Columns) { |
| 586 | // This word fits on the current line; print it there. |
| 587 | if (WordStart) { |
| 588 | OS << ' '; |
| 589 | Column += 1; |
| 590 | } |
| 591 | OS.write(&Str[WordStart], WordLength); |
| 592 | Column += WordLength; |
| 593 | continue; |
| 594 | } |
| 595 | |
| 596 | // This word does not fit on the current line, so wrap to the next |
| 597 | // line. |
Douglas Gregor | 6f5a38b | 2009-05-03 03:52:38 +0000 | [diff] [blame] | 598 | OS << '\n'; |
| 599 | OS.write(&IndentStr[0], Indentation); |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 600 | OS.write(&Str[WordStart], WordLength); |
| 601 | Column = Indentation + WordLength; |
| 602 | Wrapped = true; |
| 603 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 604 | |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 605 | if (Length == Str.size()) |
| 606 | return Wrapped; // We're done. |
| 607 | |
| 608 | // There is a newline in the message, followed by something that |
| 609 | // will not be word-wrapped. Print that. |
| 610 | OS.write(&Str[Length], Str.size() - Length); |
| 611 | return true; |
| 612 | } |
Chris Lattner | 2cf68c1 | 2009-02-17 07:38:37 +0000 | [diff] [blame] | 613 | |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 614 | void TextDiagnosticPrinter::HandleDiagnostic(Diagnostic::Level Level, |
Chris Lattner | 8488c82 | 2008-11-18 07:04:44 +0000 | [diff] [blame] | 615 | const DiagnosticInfo &Info) { |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 616 | // Keeps track of the the starting position of the location |
| 617 | // information (e.g., "foo.c:10:4:") that precedes the error |
| 618 | // message. We use this information to determine how long the |
| 619 | // file+line+column number prefix is. |
| 620 | uint64_t StartOfLocationInfo = OS.tell(); |
| 621 | |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 622 | // If the location is specified, print out a file/line/col and include trace |
| 623 | // if enabled. |
| 624 | if (Info.getLocation().isValid()) { |
Ted Kremenek | 5241832 | 2009-01-28 20:47:47 +0000 | [diff] [blame] | 625 | const SourceManager &SM = Info.getLocation().getManager(); |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 626 | PresumedLoc PLoc = SM.getPresumedLoc(Info.getLocation()); |
| 627 | unsigned LineNo = PLoc.getLine(); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 628 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 629 | // First, if this diagnostic is not in the main file, print out the |
| 630 | // "included from" lines. |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 631 | if (LastWarningLoc != PLoc.getIncludeLoc()) { |
| 632 | LastWarningLoc = PLoc.getIncludeLoc(); |
| 633 | PrintIncludeStack(LastWarningLoc, SM); |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 634 | StartOfLocationInfo = OS.tell(); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 635 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 636 | |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 637 | // Compute the column number. |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 638 | if (DiagOpts->ShowLocation) { |
| 639 | if (DiagOpts->ShowColors) |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 640 | OS.changeColor(savedColor, true); |
Chris Lattner | 1958677 | 2009-01-30 17:41:53 +0000 | [diff] [blame] | 641 | OS << PLoc.getFilename() << ':' << LineNo << ':'; |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 642 | if (DiagOpts->ShowColumn) |
Chris Lattner | 11a2a43 | 2009-02-17 07:34:34 +0000 | [diff] [blame] | 643 | if (unsigned ColNo = PLoc.getColumn()) |
| 644 | OS << ColNo << ':'; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 645 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 646 | if (DiagOpts->ShowSourceRanges && Info.getNumRanges()) { |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 647 | FileID CaretFileID = |
| 648 | SM.getFileID(SM.getInstantiationLoc(Info.getLocation())); |
| 649 | bool PrintedRange = false; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 650 | |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 651 | for (unsigned i = 0, e = Info.getNumRanges(); i != e; ++i) { |
Chris Lattner | 3251e3c | 2009-04-19 22:24:10 +0000 | [diff] [blame] | 652 | // Ignore invalid ranges. |
| 653 | if (!Info.getRange(i).isValid()) continue; |
| 654 | |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 655 | SourceLocation B = Info.getRange(i).getBegin(); |
| 656 | SourceLocation E = Info.getRange(i).getEnd(); |
Chris Lattner | 6ed7d59 | 2009-06-15 05:18:27 +0000 | [diff] [blame] | 657 | B = SM.getInstantiationLoc(B); |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 658 | E = SM.getInstantiationLoc(E); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 659 | |
Chris Lattner | 6ed7d59 | 2009-06-15 05:18:27 +0000 | [diff] [blame] | 660 | // If the End location and the start location are the same and are a |
| 661 | // macro location, then the range was something that came from a macro |
| 662 | // expansion or _Pragma. If this is an object-like macro, the best we |
| 663 | // can do is to highlight the range. If this is a function-like |
| 664 | // macro, we'd also like to highlight the arguments. |
| 665 | if (B == E && Info.getRange(i).getEnd().isMacroID()) |
| 666 | E = SM.getInstantiationRange(Info.getRange(i).getEnd()).second; |
| 667 | |
| 668 | std::pair<FileID, unsigned> BInfo = SM.getDecomposedLoc(B); |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 669 | std::pair<FileID, unsigned> EInfo = SM.getDecomposedLoc(E); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 670 | |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 671 | // If the start or end of the range is in another file, just discard |
| 672 | // it. |
| 673 | if (BInfo.first != CaretFileID || EInfo.first != CaretFileID) |
| 674 | continue; |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 675 | |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 676 | // Add in the length of the token, so that we cover multi-char tokens. |
Chris Lattner | 184e65d | 2009-04-14 23:22:57 +0000 | [diff] [blame] | 677 | unsigned TokSize = Lexer::MeasureTokenLength(E, SM, *LangOpts); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 678 | |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 679 | OS << '{' << SM.getLineNumber(BInfo.first, BInfo.second) << ':' |
| 680 | << SM.getColumnNumber(BInfo.first, BInfo.second) << '-' |
| 681 | << SM.getLineNumber(EInfo.first, EInfo.second) << ':' |
| 682 | << (SM.getColumnNumber(EInfo.first, EInfo.second)+TokSize) << '}'; |
| 683 | PrintedRange = true; |
| 684 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 685 | |
Chris Lattner | 44219f3 | 2009-03-13 01:08:23 +0000 | [diff] [blame] | 686 | if (PrintedRange) |
| 687 | OS << ':'; |
| 688 | } |
Chris Lattner | 1958677 | 2009-01-30 17:41:53 +0000 | [diff] [blame] | 689 | OS << ' '; |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 690 | if (DiagOpts->ShowColors) |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 691 | OS.resetColor(); |
| 692 | } |
| 693 | } |
| 694 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 695 | if (DiagOpts->ShowColors) { |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 696 | // Print diagnostic category in bold and color |
| 697 | switch (Level) { |
| 698 | case Diagnostic::Ignored: assert(0 && "Invalid diagnostic type"); |
| 699 | case Diagnostic::Note: OS.changeColor(noteColor, true); break; |
| 700 | case Diagnostic::Warning: OS.changeColor(warningColor, true); break; |
| 701 | case Diagnostic::Error: OS.changeColor(errorColor, true); break; |
| 702 | case Diagnostic::Fatal: OS.changeColor(fatalColor, true); break; |
Chris Lattner | 1958677 | 2009-01-30 17:41:53 +0000 | [diff] [blame] | 703 | } |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 704 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 705 | |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 706 | switch (Level) { |
Chris Lattner | b05f49e | 2009-02-06 03:57:44 +0000 | [diff] [blame] | 707 | case Diagnostic::Ignored: assert(0 && "Invalid diagnostic type"); |
Nate Begeman | 36c4927 | 2008-04-17 18:06:57 +0000 | [diff] [blame] | 708 | case Diagnostic::Note: OS << "note: "; break; |
| 709 | case Diagnostic::Warning: OS << "warning: "; break; |
| 710 | case Diagnostic::Error: OS << "error: "; break; |
Chris Lattner | b05f49e | 2009-02-06 03:57:44 +0000 | [diff] [blame] | 711 | case Diagnostic::Fatal: OS << "fatal error: "; break; |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 712 | } |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 713 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 714 | if (DiagOpts->ShowColors) |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 715 | OS.resetColor(); |
| 716 | |
Chris Lattner | 23be067 | 2008-11-19 06:51:40 +0000 | [diff] [blame] | 717 | llvm::SmallString<100> OutStr; |
| 718 | Info.FormatDiagnostic(OutStr); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 719 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 720 | if (DiagOpts->ShowOptionNames) |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 721 | if (const char *Opt = Diagnostic::getWarningOptionForDiag(Info.getID())) { |
| 722 | OutStr += " [-W"; |
| 723 | OutStr += Opt; |
| 724 | OutStr += ']'; |
| 725 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 726 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 727 | if (DiagOpts->ShowColors) { |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 728 | // Print warnings, errors and fatal errors in bold, no color |
| 729 | switch (Level) { |
| 730 | case Diagnostic::Warning: OS.changeColor(savedColor, true); break; |
| 731 | case Diagnostic::Error: OS.changeColor(savedColor, true); break; |
| 732 | case Diagnostic::Fatal: OS.changeColor(savedColor, true); break; |
| 733 | default: break; //don't bold notes |
| 734 | } |
| 735 | } |
| 736 | |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 737 | if (DiagOpts->MessageLength) { |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 738 | // We will be word-wrapping the error message, so compute the |
| 739 | // column number where we currently are (after printing the |
| 740 | // location information). |
| 741 | unsigned Column = OS.tell() - StartOfLocationInfo; |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 742 | PrintWordWrapped(OS, OutStr, DiagOpts->MessageLength, Column); |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 743 | } else { |
| 744 | OS.write(OutStr.begin(), OutStr.size()); |
| 745 | } |
Chris Lattner | 23be067 | 2008-11-19 06:51:40 +0000 | [diff] [blame] | 746 | OS << '\n'; |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 747 | if (DiagOpts->ShowColors) |
Torok Edwin | c91b6e0 | 2009-06-04 07:18:23 +0000 | [diff] [blame] | 748 | OS.resetColor(); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 749 | |
Douglas Gregor | 79cf603 | 2009-03-10 20:44:00 +0000 | [diff] [blame] | 750 | // If caret diagnostics are enabled and we have location, we want to |
| 751 | // emit the caret. However, we only do this if the location moved |
| 752 | // from the last diagnostic, if the last diagnostic was a note that |
| 753 | // was part of a different warning or error diagnostic, or if the |
| 754 | // diagnostic has ranges. We don't want to emit the same caret |
| 755 | // multiple times if one loc has multiple diagnostics. |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 756 | if (DiagOpts->ShowCarets && Info.getLocation().isValid() && |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 757 | ((LastLoc != Info.getLocation()) || Info.getNumRanges() || |
Douglas Gregor | 79cf603 | 2009-03-10 20:44:00 +0000 | [diff] [blame] | 758 | (LastCaretDiagnosticWasNote && Level != Diagnostic::Note) || |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 759 | Info.getNumCodeModificationHints())) { |
Steve Naroff | a3699224 | 2008-02-08 22:06:17 +0000 | [diff] [blame] | 760 | // Cache the LastLoc, it allows us to omit duplicate source/caret spewage. |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 761 | LastLoc = Info.getLocation(); |
Douglas Gregor | 79cf603 | 2009-03-10 20:44:00 +0000 | [diff] [blame] | 762 | LastCaretDiagnosticWasNote = (Level == Diagnostic::Note); |
Chris Lattner | f1ca7d3 | 2009-01-27 07:57:44 +0000 | [diff] [blame] | 763 | |
Chris Lattner | 1973d84 | 2009-02-20 00:18:51 +0000 | [diff] [blame] | 764 | // Get the ranges into a local array we can hack on. |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 765 | SourceRange Ranges[20]; |
Chris Lattner | 1973d84 | 2009-02-20 00:18:51 +0000 | [diff] [blame] | 766 | unsigned NumRanges = Info.getNumRanges(); |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 767 | assert(NumRanges < 20 && "Out of space"); |
Chris Lattner | 1973d84 | 2009-02-20 00:18:51 +0000 | [diff] [blame] | 768 | for (unsigned i = 0; i != NumRanges; ++i) |
| 769 | Ranges[i] = Info.getRange(i); |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 770 | |
Douglas Gregor | 87f95b0 | 2009-02-26 21:00:50 +0000 | [diff] [blame] | 771 | unsigned NumHints = Info.getNumCodeModificationHints(); |
| 772 | for (unsigned idx = 0; idx < NumHints; ++idx) { |
| 773 | const CodeModificationHint &Hint = Info.getCodeModificationHint(idx); |
| 774 | if (Hint.RemoveRange.isValid()) { |
| 775 | assert(NumRanges < 20 && "Out of space"); |
| 776 | Ranges[NumRanges++] = Hint.RemoveRange; |
| 777 | } |
| 778 | } |
| 779 | |
| 780 | EmitCaretDiagnostic(LastLoc, Ranges, NumRanges, LastLoc.getManager(), |
| 781 | Info.getCodeModificationHints(), |
Douglas Gregor | 4818553 | 2009-05-01 21:53:04 +0000 | [diff] [blame] | 782 | Info.getNumCodeModificationHints(), |
Daniel Dunbar | c2e6a47 | 2009-11-04 06:24:30 +0000 | [diff] [blame] | 783 | DiagOpts->MessageLength); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 784 | } |
Daniel Dunbar | 49f0e80 | 2009-09-07 23:07:56 +0000 | [diff] [blame] | 785 | |
Chris Lattner | 327984f | 2008-11-19 06:56:25 +0000 | [diff] [blame] | 786 | OS.flush(); |
Bill Wendling | 37b1dde | 2007-06-07 09:34:54 +0000 | [diff] [blame] | 787 | } |