Didn't notice that yield an error, thanks again Sean! On Wed, Apr 6, 2016 at 7:04 PM, Sean Silva <chisophu...@gmail.com> wrote:
> > > On Wed, Apr 6, 2016 at 6:56 PM, Sean Silva <chisophu...@gmail.com> wrote: > >> >> >> On Wed, Apr 6, 2016 at 6:10 PM, Bruno Cardoso Lopes via cfe-commits < >> cfe-commits@lists.llvm.org> wrote: >> >>> Attempt to fix in r265630. >>> >> >> Did you ever fix PATH_MAX? >> > > I've fixed it in r265634. > > -- Sean Silva > > >> >>> >>> On Wed, Apr 6, 2016 at 5:51 PM, Bruno Cardoso Lopes < >>> bruno.card...@gmail.com> wrote: >>> >>>> Thanks Hans, taking a look! >>>> >>>> On Wed, Apr 6, 2016 at 5:49 PM, Hans Wennborg <h...@chromium.org> >>>> wrote: >>>> >>>>> On Wed, Apr 6, 2016 at 5:00 PM, Bruno Cardoso Lopes via cfe-commits >>>>> <cfe-commits@lists.llvm.org> wrote: >>>>> > Author: bruno >>>>> > Date: Wed Apr 6 19:00:57 2016 >>>>> > New Revision: 265622 >>>>> > >>>>> > URL: http://llvm.org/viewvc/llvm-project?rev=265622&view=rev >>>>> > Log: >>>>> > [CrashReproducer] Setup 'case-sensitive' in YAML files. >>>>> >>>>> This doesn't compile on Windows: >>>>> >>>>> >>>>> C:\buildbot\slave-config\clang-x86-win2008-selfhost\llvm\tools\clang\lib\Frontend\ModuleDependencyCollector.cpp(83) >>>>> : error C2065: 'PATH_MAX' : undeclared identifier >>>>> >>>>> C:\buildbot\slave-config\clang-x86-win2008-selfhost\llvm\tools\clang\lib\Frontend\ModuleDependencyCollector.cpp(83) >>>>> : error C2975: 'InternalLen' : invalid template argument for >>>>> 'llvm::SmallString', expected compile-time constant expression >>>>> >>>>> C:\buildbot\slave-config\clang-x86-win2008-selfhost\llvm\include\llvm/ADT/SmallString.h(24) >>>>> : see declaration of 'InternalLen' >>>>> >>>>> C:\buildbot\slave-config\clang-x86-win2008-selfhost\llvm\tools\clang\lib\Frontend\ModuleDependencyCollector.cpp(94) >>>>> : error C2039: 'toupper' : is not a member of 'std' >>>>> ninja: build stopped: subcommand failed. >>>>> >>>>> From >>>>> http://lab.llvm.org:8011/builders/clang-x86-win2008-selfhost/builds/7340/steps/build%20stage%201/logs/stdio >>>>> >>>> >>>> >>>> >>>> -- >>>> Bruno Cardoso Lopes >>>> http://www.brunocardoso.cc >>>> >>> >>> >>> >>> -- >>> Bruno Cardoso Lopes >>> http://www.brunocardoso.cc >>> >>> _______________________________________________ >>> cfe-commits mailing list >>> cfe-commits@lists.llvm.org >>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >>> >>> >> > -- Bruno Cardoso Lopes http://www.brunocardoso.cc
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
