[cfe-commits] [PATCH] test 'nodebug' on method

Robinson, Paul Paul.Robinson at am.sony.com
Tue Oct 23 14:46:17 PDT 2012


> > 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
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: diff-nodebug-Sema.txt
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121023/716ed428/attachment.txt>


More information about the cfe-commits mailing list