> > Because I keep seeing comments on the lists about not creating trivial
> > new test files, instead we should combine tests when it makes sense.
> > This test is all about the 'nodebug' attribute so all those tests
> > belong in one file. My new piece is specific to C++ so the test needed
> > to be turned into a C++ source.
>
> Sure, but C and C++ are different, even if we try to share as much of
> Sema as possible. So in this case I think it is probably better to
> just add the c++ case in a new file.
Okay, itty bitty new C++ only test patch attached.
--paulr
Index: test/Sema/attr-nodebug.cpp
===================================================================
--- test/Sema/attr-nodebug.cpp (revision 0)
+++ test/Sema/attr-nodebug.cpp (revision 0)
@@ -0,0 +1,7 @@
+// RUN: %clang_cc1 %s -verify -fsyntax-only
+// Note: most of the 'nodebug' tests are in attr-nodebug.c.
+
+// expected-no-diagnostics
+class c {
+ void t3() __attribute__((nodebug));
+};
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
