[PATCH] D51111: Add more pre-run asserts for the DirCompletionAbsolute test

Stella Stamenova via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 24 13:36:35 PDT 2018


stella.stamenova added a comment.

This does not build correctly on Windows (at least not in our environment). Here are the two errors:

[error]llvm\tools\lldb\unittests\interpreter\testcompletion.cpp(168,0): Error C2065: 'PATH_MAX': undeclared identifier
----------------------------------------------------------------------------------------------------------------------

[error]llvm\tools\lldb\unittests\interpreter\testcompletion.cpp(168,0): Error C2512: 'testing::AssertionResult': no appropriate default constructor available
-------------------------------------------------------------------------------------------------------------------------------------------------------------

I haven't had an opportunity to investigate further yet.


Repository:
  rL LLVM

https://reviews.llvm.org/D51111





More information about the llvm-commits mailing list