r244427 - clang/test/Modules: Satisfy win32 hosts due to dospath issue.

Yaron Keren via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 9 23:25:14 PDT 2015


This issue is surfacing regulary. In cases where clang prints the escaped
path we have to use {{/\\\\} which is not very readable, for example in
mingw driver case:

// CHECK_MINGW_ORG_TREE:
"{{.*}}/Inputs/mingw_mingw_org_tree/mingw{{/|\\\\}}lib{{/|\\\\}}gcc{{/|\\\\}}mingw32{{/|\\\\}}4.8.1{{/|\\\\}}include{{/|\\\\}}c++"

Could we allow FileCheck to always accept \  and \\ as matches for / or
would that too sloppy?


2015-08-10 5:56 GMT+03:00 Richard Smith via cfe-commits <
cfe-commits at lists.llvm.org>:

> On Aug 9, 2015 3:56 PM, "NAKAMURA Takumi via cfe-commits" <
> cfe-commits at lists.llvm.org> wrote:
> >
> > Author: chapuni
> > Date: Sun Aug  9 17:55:50 2015
> > New Revision: 244427
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=244427&view=rev
> > Log:
> > clang/test/Modules: Satisfy win32 hosts due to dospath issue.
>
> Thanks!
>
> > Modified:
> >     cfe/trunk/test/Modules/dependency-gen.modulemap
> >     cfe/trunk/test/Modules/explicit-build.cpp
> >     cfe/trunk/test/Modules/relative-dep-gen.cpp
> >
> > Modified: cfe/trunk/test/Modules/dependency-gen.modulemap
> > URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/dependency-gen.modulemap?rev=244427&r1=244426&r2=244427&view=diff
> >
> ==============================================================================
> > --- cfe/trunk/test/Modules/dependency-gen.modulemap (original)
> > +++ cfe/trunk/test/Modules/dependency-gen.modulemap Sun Aug  9 17:55:50
> 2015
> > @@ -39,5 +39,5 @@ extern module "test-base" "Inputs/depend
> >  // EXPLICIT-FIXME: {{.*}}/base.pcm
> >  //
> >  // EXPLICIT: {{^}}explicit.pcm:
> > -// EXPLICIT: {{.*}}/dependency-gen.modulemap
> > +// EXPLICIT: {{.*[/\\]}}dependency-gen.modulemap
> >  // EXPLICIT: {{ |\.[/\\]}}Inputs{{[/\\]}}dependency-gen.h
> >
> > Modified: cfe/trunk/test/Modules/explicit-build.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/explicit-build.cpp?rev=244427&r1=244426&r2=244427&view=diff
> >
> ==============================================================================
> > --- cfe/trunk/test/Modules/explicit-build.cpp (original)
> > +++ cfe/trunk/test/Modules/explicit-build.cpp Sun Aug  9 17:55:50 2015
> > @@ -138,7 +138,7 @@
> >  // RUN:
> -fmodule-map-file=%S/Inputs/explicit-build/module.modulemap \
> >  // RUN:            %s 2>&1 | FileCheck --check-prefix=CHECK-MULTIPLE-AS
> %s
> >  //
> > -// CHECK-MULTIPLE-AS: error: module 'a' is defined in both
> '{{.*}}/a{{.*}}.pcm' and '{{.*[/\\]}}a{{.*}}.pcm'
> > +// CHECK-MULTIPLE-AS: error: module 'a' is defined in both
> '{{.*[/\\]}}a{{.*}}.pcm' and '{{.*[/\\]}}a{{.*}}.pcm'
> >
> >  // -------------------------------
> >  // Try to import a PCH with -fmodule-file=
> >
> > Modified: cfe/trunk/test/Modules/relative-dep-gen.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/relative-dep-gen.cpp?rev=244427&r1=244426&r2=244427&view=diff
> >
> ==============================================================================
> > --- cfe/trunk/test/Modules/relative-dep-gen.cpp (original)
> > +++ cfe/trunk/test/Modules/relative-dep-gen.cpp Sun Aug  9 17:55:50 2015
> > @@ -27,4 +27,4 @@
> >  // CHECK-USE: use.o:
> >  // CHECK-USE-DAG:   Inputs/relative-dep-gen{{(-cwd)?}}.modulemap
>
> Is this one OK?
>
> >  // CHECK-USE-DAG:   relative-dep-gen.cpp
> > -// CHECK-USE-DAG:   Inputs/relative-dep-gen-1.h
> > +// CHECK-USE-DAG:   Inputs{{[/\\]}}relative-dep-gen-1.h
> >
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at lists.llvm.org
> > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150810/a9b36b55/attachment.html>


More information about the cfe-commits mailing list