r223935 - Added a testcase to make sure the parser allows

Jordan Rose jordan_rose at apple.com
Wed Dec 10 17:02:11 PST 2014


That's not actually a valid statement (it needs a semicolon). Why do we allow this?

> On Dec 10, 2014, at 10:57, Sean Callanan <scallanan at apple.com> wrote:
> 
> Author: spyffe
> Date: Wed Dec 10 12:57:09 2014
> New Revision: 223935
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=223935&view=rev
> Log:
> Added a testcase to make sure the parser allows
> but ignores module imports in debugger mode,
> even inside functions.
> 
> Added:
>    cfe/trunk/test/Parser/debugger-import-module.m
> 
> Added: cfe/trunk/test/Parser/debugger-import-module.m
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Parser/debugger-import-module.m?rev=223935&view=auto
> ==============================================================================
> --- cfe/trunk/test/Parser/debugger-import-module.m (added)
> +++ cfe/trunk/test/Parser/debugger-import-module.m Wed Dec 10 12:57:09 2014
> @@ -0,0 +1,6 @@
> +// RUN: %clang_cc1 -fsyntax-only -fdebugger-support -verify %s
> +// expected-no-diagnostics
> +
> +void importAModule() {
> +  @import AModuleThatDoesntExist
> +}
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits





More information about the cfe-commits mailing list