Re: r299410 - Modules/builtins.m requires shell.

2017-04-03 Thread Galina Kistanova via cfe-commits
Just wanted temporarily disable this test on Windows bots while investigating the clang assert which happens once in a while By the way, do you see something like this on your bots? Assertion failed: (Imported == nullptr || Imported == SuggestedModule.getModule()) && "the imported module is differ

Re: r299410 - Modules/builtins.m requires shell.

2017-04-03 Thread NAKAMURA Takumi via cfe-commits
What was your issue? Seems nothing to do "shell". On Tue, Apr 4, 2017 at 12:03 PM Galina Kistanova via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: gkistanova > Date: Mon Apr 3 21:50:40 2017 > New Revision: 299410 > > URL: http://llvm.org/viewvc/llvm-project?rev=299410&view=rev > L

r299410 - Modules/builtins.m requires shell.

2017-04-03 Thread Galina Kistanova via cfe-commits
Author: gkistanova Date: Mon Apr 3 21:50:40 2017 New Revision: 299410 URL: http://llvm.org/viewvc/llvm-project?rev=299410&view=rev Log: Modules/builtins.m requires shell. Modified: cfe/trunk/test/Modules/builtins.m Modified: cfe/trunk/test/Modules/builtins.m URL: http://llvm.org/viewvc/llv