summaryrefslogtreecommitdiffstats
path: root/clang/test/FixIt/fixit.c
Commit message (Collapse)AuthorAgeFilesLines
* [clang] Cleanup fixit.cAlexander Shaposhnikov2017-06-091-2/+1
| | | | | | | | | | | | This diff removes temporary file t2 in fixit.c and updates the test command accordingly. NFC. Test plan: make check-all Differential revision: https://reviews.llvm.org/D34066 llvm-svn: 305124
* Unify handling of string literal arguments for attributes and add fixits.Benjamin Kramer2013-09-131-0/+2
| | | | | | | | | | | | | | | | | | | | | | This fixes a couple of latent crashes for invalid attributes and also adds a fixit hint to turn identifiers into string literals if one was expected. It then proceeds recovery as if the identifier was a literal. Diagnostic locations are also changed to point at the literal instead of the attribute if the error concerns the argument. PR17175. For example: hidden.c:1:40: error: 'visibility' attribute requires a string extern int x __attribute__((visibility(hidden))); ^ " " hidden.c:2:29: error: visibility does not match previous declaration extern int x __attribute__((visibility("default"))); ^ hidden.c:1:29: note: previous attribute is here extern int x __attribute__((visibility(hidden))); ^ llvm-svn: 190699
* If a comma operator is followed by a token which unambiguously indicates theRichard Smith2012-09-181-0/+7
| | | | | | | | start of a statement or the end of a compound-statement, diagnose the comma as a typo for a semicolon. Patch by Ahmed Bougacha! Additional test cases and minor refactoring by me. llvm-svn: 164085
* Extend the "expected ';' after struct" logic to also apply to enums, and toRichard Smith2012-06-251-0/+8
| | | | | | struct and enum forward-declarations. llvm-svn: 159164
* Reject continue/break statements within members of local functions nested withinRichard Smith2012-02-171-19/+42
| | | | | | | | | | | | | | | loop and switch statements, by teaching Scope that a function scope never has a continue/break parent for the purposes of control flow. Remove the hack in block and lambda expressions which worked around this by pretending that such expressions were continue/break scopes. Remove Scope::ControlParent, since it's unused. In passing, teach default statements to recover properly from a missing ';', and add a fixit for same to both default and case labels (the latter already recovered correctly). llvm-svn: 150776
* Revert most of r145372 for now. Lookahead beyond the ';' in a functionRichard Smith2011-11-301-8/+0
| | | | | | | | | declaration tickles a bug in the way we handle visibility pragmas. The improvement to error recovery for template function definitions declared with the 'typedef' specifier in r145372 is unrelated and not reverted here. llvm-svn: 145541
* PR10101: Recover better from a common copy-paste error: if a functionRichard Smith2011-11-291-0/+8
| | | | | | | | declaration at namespace scope is followed by a semicolon and an open-brace (or in C++, a 'try', ':' or '='), then the error is probably a function definition with a spurious ';', rather than a mysterious '{'. llvm-svn: 145372
* Improve the diagnostic when a comma ends up at the end of a declarator groupRichard Smith2011-10-191-0/+7
| | | | | | | | | instead of a semicolon (as sometimes happens during refactorings). When such a comma is seen at the end of a line, and is followed by something which can't possibly be a declarator (or even something which might be a plausible typo for a declarator), suggest that a semicolon was intended. llvm-svn: 142544
* PR10867: Work around a bug in lit. Multiple RUN: lines are joined with &&, so:Richard Smith2011-09-061-1/+1
| | | | | | | | | | | | | | | | | RUN: foo RUN: bar || true is equivalent to: RUN: foo && bar || true which is equivalent to: RUN: (foo && bar) || true This resulted in several of the fixit tests not really testing anything. llvm-svn: 139132
* Add a fixit for removal of unused label.Anna Zaks2011-07-281-2/+13
| | | | llvm-svn: 136389
* Allow the fixit for missing ':' in the ?: ternary operator if it is pointingArgyrios Kyrtzidis2011-06-241-0/+5
| | | | | | at the start of a macro instantiation. llvm-svn: 133801
* Fixit suggestion for adding missing tag name should have a space after the ↵Argyrios Kyrtzidis2011-04-211-0/+6
| | | | | | tag name. Fixes rdar://9295072 llvm-svn: 129917
* Improve recovery when a comma is missing between enumerators in anDouglas Gregor2010-09-071-0/+7
| | | | | | enumeration definition. Fixes <rdar://problem/7159693>. llvm-svn: 113201
* When complaining about a duplicate declspec, provide a Fix-It thatDouglas Gregor2010-08-231-0/+3
| | | | | | | removes the copy. Patch from Eelis van der Weegen, tweaked/updated by me. llvm-svn: 111807
* Teach clang -fixit to modify files in-place, or -fixit=suffix to create newNick Lewycky2010-04-241-4/+6
| | | | | | files with the additional suffix in the middle. llvm-svn: 102230
* fix the ?: fixit that ted added to recover properly.Chris Lattner2010-04-201-3/+3
| | | | llvm-svn: 101943
* Add fixit hint for missing ':' in ternary expressions.Ted Kremenek2010-04-121-0/+5
| | | | llvm-svn: 101073
* Remove fixit for string literal comparison. Telling the user to use ↵Ted Kremenek2010-04-091-6/+0
| | | | | | | | | 'strcmp' is bad, and we don't have enough information to tell them how to use 'strncmp'. Instead, change the diagnostic to indicate they should use 'strncmp'. llvm-svn: 100890
* Update tests to use %clang_cc1 instead of 'clang-cc' or 'clang -cc1'.Daniel Dunbar2009-12-151-2/+2
| | | | | | | | | - This is designed to make it obvious that %clang_cc1 is a "test variable" which is substituted. It is '%clang_cc1' instead of '%clang -cc1' because it can be useful to redefine what gets run as 'clang -cc1' (for example, to set a default target). llvm-svn: 91446
* Drop unnecessary #include.Daniel Dunbar2009-11-171-1/+3
| | | | llvm-svn: 89154
* Improve test to make sure -fixit is really working.Daniel Dunbar2009-11-141-1/+9
| | | | llvm-svn: 88801
* Update FixIt tests to make it more obvious they use a separate mode.Daniel Dunbar2009-11-141-1/+1
| | | | llvm-svn: 88758
* Fixed the Fix-It hints for comparison against a string literal. Thanks, Chris!Douglas Gregor2009-04-061-2/+2
| | | | llvm-svn: 68454
* Move the fix-it tests into their own subdirectoryDouglas Gregor2009-04-021-0/+29
llvm-svn: 68325
OpenPOWER on IntegriCloud