On Thu, Aug 29, 2013 at 6:58 PM, Peter N Lewis <[email protected]>wrote:
> On 30/08/2013, at 6:34 , Eli Friedman <[email protected]> wrote: > > +// RUNNOT: %clang_cc1 -fsyntax-only -verify %s -DZEROARGS > -Wgnu-zero-variadic-macro-arguments > > +// RUNNOT: %clang_cc1 -fsyntax-only -verify %s -DIMAGINARYCONST > -Wgnu-imaginary-constant > > +// RUNNOT: %clang_cc1 -fsyntax-only -verify %s -DBINARYLITERAL > -Wgnu-binary-literal > > +// RUNNOT: %clang_cc1 -fsyntax-only -verify %s -DLINE0 > -Wgnu-zero-line-directive > > > > I haven't seen RUNNOT before; is that some sort of new syntax? > > They are just disabled RUN statements - disabled by default to cut down on > excessive invocations of the tests. If there is a better syntax to > indicate "you could run this test if you want to, but we wont run it every > time for everyone", let me know. I initially tried // DONTRUN: but that > still runs (I guess the parser looks for RUN:). > > Please don't make it looks like a RUN command if it isn't intended to run. You can just make a list with a header like "additional tests" and have a list of unprefixed commands. > Any other issues? > > Otherwise, it's fine. -Eli
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
