Skip to content

Commit

Permalink
Add back single quotes when dontcall attribute was split into dontcal…
Browse files Browse the repository at this point in the history
…l-error/dontcall-warn

Single quotes were accidentally dropped in D110364.
  • Loading branch information
MaskRay committed Nov 12, 2022
1 parent 8b3f783 commit e588d23
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 21 deletions.
4 changes: 2 additions & 2 deletions clang/include/clang/Basic/DiagnosticFrontendKinds.td
Expand Up @@ -86,9 +86,9 @@ def err_fe_backend_unsupported : Error<"%0">, BackendInfo;
def warn_fe_backend_unsupported : Warning<"%0">, BackendInfo;

def err_fe_backend_error_attr :
Error<"call to %0 declared with 'error' attribute: %1">, BackendInfo;
Error<"call to '%0' declared with 'error' attribute: %1">, BackendInfo;
def warn_fe_backend_warning_attr :
Warning<"call to %0 declared with 'warning' attribute: %1">, BackendInfo,
Warning<"call to '%0' declared with 'warning' attribute: %1">, BackendInfo,
InGroup<BackendWarningAttributes>;

def err_fe_invalid_code_complete_file : Error<
Expand Down
Expand Up @@ -8,7 +8,7 @@ int x(void) {
return 8 % 2 == 1;
}
void baz(void) {
foo(); // expected-error {{call to foo declared with 'error' attribute: oh no foo}}
foo(); // expected-error {{call to 'foo' declared with 'error' attribute: oh no foo}}
if (x())
bar();
}
Expand Down
12 changes: 6 additions & 6 deletions clang/test/Frontend/backend-attribute-error-warning.c
Expand Up @@ -22,12 +22,12 @@ void // expected-note@-1 {{previous a
duplicate_warnings(void);

void baz(void) {
foo(); // expected-error {{call to foo declared with 'error' attribute: oh no foo}}
foo(); // expected-error {{call to 'foo' declared with 'error' attribute: oh no foo}}
if (x())
bar(); // expected-error {{call to bar declared with 'error' attribute: oh no bar}}
bar(); // expected-error {{call to 'bar' declared with 'error' attribute: oh no bar}}

quux(); // enabled-warning {{call to quux declared with 'warning' attribute: oh no quux}}
__compiletime_assert_455(); // expected-error {{call to __compiletime_assert_455 declared with 'error' attribute: demangle me}}
duplicate_errors(); // expected-error {{call to duplicate_errors declared with 'error' attribute: two}}
duplicate_warnings(); // enabled-warning {{call to duplicate_warnings declared with 'warning' attribute: two}}
quux(); // enabled-warning {{call to 'quux' declared with 'warning' attribute: oh no quux}}
__compiletime_assert_455(); // expected-error {{call to '__compiletime_assert_455' declared with 'error' attribute: demangle me}}
duplicate_errors(); // expected-error {{call to 'duplicate_errors' declared with 'error' attribute: two}}
duplicate_warnings(); // enabled-warning {{call to 'duplicate_warnings' declared with 'warning' attribute: two}}
}
24 changes: 12 additions & 12 deletions clang/test/Frontend/backend-attribute-error-warning.cpp
Expand Up @@ -23,14 +23,14 @@ void // expected-note@-1 {{previous a
duplicate_warnings(void);

void baz(void) {
foo(); // expected-error {{call to foo() declared with 'error' attribute: oh no foo}}
foo(); // expected-error {{call to 'foo()' declared with 'error' attribute: oh no foo}}
if (x())
bar(); // expected-error {{call to bar() declared with 'error' attribute: oh no bar}}
bar(); // expected-error {{call to 'bar()' declared with 'error' attribute: oh no bar}}

quux(); // enabled-warning {{call to quux() declared with 'warning' attribute: oh no quux}}
__compiletime_assert_455(); // expected-error {{call to __compiletime_assert_455() declared with 'error' attribute: demangle me}}
duplicate_errors(); // expected-error {{call to duplicate_errors() declared with 'error' attribute: two}}
duplicate_warnings(); // enabled-warning {{call to duplicate_warnings() declared with 'warning' attribute: two}}
quux(); // enabled-warning {{call to 'quux()' declared with 'warning' attribute: oh no quux}}
__compiletime_assert_455(); // expected-error {{call to '__compiletime_assert_455()' declared with 'error' attribute: demangle me}}
duplicate_errors(); // expected-error {{call to 'duplicate_errors()' declared with 'error' attribute: two}}
duplicate_warnings(); // enabled-warning {{call to 'duplicate_warnings()' declared with 'warning' attribute: two}}
}

#ifdef __cplusplus
Expand All @@ -45,16 +45,16 @@ struct Widget {
};

void baz_cpp(void) {
foo(); // expected-error {{call to foo() declared with 'error' attribute: oh no foo}}
foo(); // expected-error {{call to 'foo()' declared with 'error' attribute: oh no foo}}
if (x())
bar(); // expected-error {{call to bar() declared with 'error' attribute: oh no bar}}
quux(); // enabled-warning {{call to quux() declared with 'warning' attribute: oh no quux}}
bar(); // expected-error {{call to 'bar()' declared with 'error' attribute: oh no bar}}
quux(); // enabled-warning {{call to 'quux()' declared with 'warning' attribute: oh no quux}}

// Test that we demangle correctly in the diagnostic for C++.
__compiletime_assert_455(); // expected-error {{call to __compiletime_assert_455() declared with 'error' attribute: demangle me}}
nocall<int>(42); // expected-error {{call to int nocall<int>(int) declared with 'error' attribute: demangle me, too}}
__compiletime_assert_455(); // expected-error {{call to '__compiletime_assert_455()' declared with 'error' attribute: demangle me}}
nocall<int>(42); // expected-error {{call to 'int nocall<int>(int)' declared with 'error' attribute: demangle me, too}}

Widget W;
int w = W; // enabled-warning {{Widget::operator int() declared with 'warning' attribute: don't call me!}}
int w = W; // enabled-warning {{call to 'Widget::operator int()' declared with 'warning' attribute: don't call me!}}
}
#endif

0 comments on commit e588d23

Please sign in to comment.