test/Driver/cl.c, remove quotes around /help

The quotes (from r187330) didn't really help here, the trick was to disable
the test on MSYS builds. This removes those quotes, changes back the comment
to explain why /? has to be quoted specifically, and moves the REQUIRES
line to the top of the file because that's important.

llvm-svn: 187366
This commit is contained in:
Hans Wennborg 2013-07-29 18:40:38 +00:00
parent 021221db7c
commit 4a9ca978b0
1 changed files with 11 additions and 10 deletions

View File

@ -1,18 +1,22 @@
// Note: we have to quote the options with a /, otherwise some shells will try
// to expand it and make test fail.
// Don't attempt slash switches on msys bash.
// REQUIRES: shell-preserves-root
// Note: we have to quote the /? option, otherwise some shells will try to
// expand the ? into a one-letter filename in the root directory, and make
// the test fail is such a file or directory exists.
// Check that clang-cl options are not available by default.
// RUN: %clang -help | FileCheck %s -check-prefix=DEFAULT
// DEFAULT-NOT: CL.EXE COMPATIBILITY OPTIONS
// DEFAULT-NOT: {{/[?]}}
// DEFAULT-NOT: /help
// RUN: not %clang '/?'
// RUN: not %clang "/?"
// RUN: not %clang -?
// RUN: not %clang '/help'
// RUN: not %clang /help
// Check that '/?' and '/help' are available as clang-cl options.
// RUN: %clang_cl '/?' | FileCheck %s -check-prefix=CL
// RUN: %clang_cl '/help' | FileCheck %s -check-prefix=CL
// Check that /? and /help are available as clang-cl options.
// RUN: %clang_cl "/?" | FileCheck %s -check-prefix=CL
// RUN: %clang_cl /help | FileCheck %s -check-prefix=CL
// RUN: %clang_cl -help | FileCheck %s -check-prefix=CL
// CL: CL.EXE COMPATIBILITY OPTIONS
// CL: {{/[?]}}
@ -22,6 +26,3 @@
// are not available in clang-cl.
// DEFAULT: -fapple-kext
// CL-NOT: -fapple-kext
// Don't attempt slash switches on msys bash.
// REQUIRES: shell-preserves-root