[test] Properly test -Werror-implicit-function-declaration and -Wvec-elem-size

[dblaikie]

Thanks for taking a look at this!

clang/test/Sema/implicit-decl.c

2

I probably wouldn't bother testing this - presumably we don't test the -Wno- variant of every warning, given that it's implemented in a generic fashion?

If this is removed, I expect the verifier prefixes don't need to be changed?

clang/test/Sema/vecshift.c

5–6

The previous version of the test had this without -DERR, right? Why the change here?

[MaskRay]

clang/test/Sema/implicit-decl.c

2

Yes. I can remove -Wno-implicit-function-declaration

clang/test/Sema/vecshift.c

5–6

I think the previous version was wrongly structured. It probably wanted to test what -Wvec-elem-size checks but Wno-error=vec-elem-size would not check anything.

[MaskRay]

comments

[MaskRay]

Discard an unintentional change

[dblaikie]

clang/test/Sema/vecshift.c

5–6

Not sure I follow - could you walk me through it in a bit more detail/few more steps?

It seems surprising to me that all 4 check lines this test has as you're proposing have -DERR, if that's correct, why have it a as a preprocessor variable (ie: all the non-ERR parts of the test are dead in this proposed version, it seems)? But the test as committed in 9941ca8af6b4c39fd0b9e47dc7e593d884b55710 tested both with -DERR and without it.

[MaskRay]

Fix vecshift.c

[MaskRay]

clang/test/Sema/vecshift.c

5–6

OK, I misunderstood the intention of the original test.

-Wvec-elem-size is an error. To test that the error can be downgraded to a warning, -Wno-error=vec-elem-size is needed.

Fixed the RUN lines.

[dblaikie]

Looks good - thanks!

clang/test/Modules/diagnose-missing-import.m

 // RUN: rm -rf %t
 // RUN: %clang_cc1 -fmodules -fmodules-cache-path=%t -I%S/Inputs/diagnose-missing-import \
 // RUN:   -Werror=implicit-function-declaration -fsyntax-only \
 // RUN:   -fimplicit-module-maps -verify %s
-// RUN: %clang_cc1 -fmodules -fmodules-cache-path=%t -I%S/Inputs/diagnose-missing-import \
-// RUN:   -Werror-implicit-function-declaration -fsyntax-only \
-// RUN:   -fimplicit-module-maps -verify %s
 @import NCI;
 
 void foo() {
   XYZLogEvent(xyzRiskyCloseOpenParam, xyzRiskyCloseOpenParam); // expected-error {{implicit declaration of function 'XYZLogEvent'}} expected-error {{declaration of 'XYZLogEvent' must be imported}} expected-error {{declaration of 'xyzRiskyCloseOpenParam' must be imported from module 'NCI.A'}} expected-error {{declaration of 'xyzRiskyCloseOpenParam' must be imported from module 'NCI.A'}}
 }
 
 // expected-note@Inputs/diagnose-missing-import/a.h:5 {{declaration here is not visible}}
 // expected-note@Inputs/diagnose-missing-import/a.h:5 {{declaration here is not visible}}
 // expected-note@Inputs/diagnose-missing-import/a.h:6 {{declaration here is not visible}}
 

clang/test/Sema/implicit-decl.c

-// RUN: %clang_cc1 %s -verify -fsyntax-only -Werror
+// RUN: %clang_cc1 %s -verify -fsyntax-only -Werror=implicit-function-declaration
+
+/// -Werror-implicit-function-declaration is a deprecated alias used by many projects.
+// RUN: %clang_cc1 %s -verify -fsyntax-only -Werror-implicit-function-declaration
 
 typedef int int32_t;
 typedef unsigned char Boolean;
 
 extern int printf(__const char *__restrict __format, ...); // expected-note{{'printf' declared here}}
 
 void func() {
    int32_t *vector[16];

clang/test/Sema/vecshift.c

 // RUN: %clang_cc1 -fsyntax-only -DERR -verify %s
+// RUN: %clang_cc1 -fsyntax-only -verify %s -Wno-error=vec-elem-size
 // RUN: %clang_cc1 -fsyntax-only -DEXT -DERR -verify %s
+// RUN: %clang_cc1 -fsyntax-only -DEXT -verify %s -Wno-error=vec-elem-size
 
 #ifdef EXT
 typedef __attribute__((__ext_vector_type__(8))) char vector_char8;
 typedef __attribute__((__ext_vector_type__(8))) short vector_short8;
 typedef __attribute__((__ext_vector_type__(8))) int vector_int8;
 typedef __attribute__((__ext_vector_type__(8))) unsigned char vector_uchar8;
 typedef __attribute__((__ext_vector_type__(8))) unsigned short vector_ushort8;
 typedef __attribute__((__ext_vector_type__(8))) unsigned int vector_uint8;
 typedef __attribute__((__ext_vector_type__(4))) char vector_char4;
 typedef __attribute__((__ext_vector_type__(4))) short vector_short4;