diff options
| author | David Malcolm <dmalcolm@redhat.com> | 2017-11-21 21:59:53 +0000 |
|---|---|---|
| committer | David Malcolm <dmalcolm@gcc.gnu.org> | 2017-11-21 21:59:53 +0000 |
| commit | d4300cc6eae1503d95eb70865f8a82adab7e552d (patch) | |
| tree | 376083f6cff4bc91793837fc6b29d03e9e4916c5 /gcc/c/c-decl.c | |
| parent | 6f9b7472a38a355e1e37d0fa35d399e693726559 (diff) | |
| download | gcc-d4300cc6eae1503d95eb70865f8a82adab7e552d.zip gcc-d4300cc6eae1503d95eb70865f8a82adab7e552d.tar.gz gcc-d4300cc6eae1503d95eb70865f8a82adab7e552d.tar.bz2 | |
C: don't suggest names that came from earlier failures (PR c/83056)
PR c/83056 reports an issue affecting trunk and gcc-7 in which
the C frontend's implementation of lookup_name_fuzzy uses undeclared
identifiers as suggestions when encountering subsequent undeclared
identifiers.
The fix is to filter out the names bound to error_mark_node
in lookup_name_fuzzy.
The C++ frontend is unaffected, as it already does this.
gcc/c/ChangeLog:
PR c/83056
* c-decl.c (lookup_name_fuzzy): Don't suggest names that came from
earlier failed lookups.
gcc/testsuite/ChangeLog:
PR c/83056
* gcc.dg/spellcheck-pr83056.c: New test case.
From-SVN: r255038
Diffstat (limited to 'gcc/c/c-decl.c')
| -rw-r--r-- | gcc/c/c-decl.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/gcc/c/c-decl.c b/gcc/c/c-decl.c index e0a4dd1..9c3beab 100644 --- a/gcc/c/c-decl.c +++ b/gcc/c/c-decl.c @@ -4035,6 +4035,8 @@ lookup_name_fuzzy (tree name, enum lookup_name_fuzzy_kind kind, location_t loc) { if (!binding->id || binding->invisible) continue; + if (binding->decl == error_mark_node) + continue; /* Don't use bindings from implicitly declared functions, as they were likely misspellings themselves. */ if (TREE_CODE (binding->decl) == FUNCTION_DECL) |