[PATCH] [Commented On] D24996: Move UTF functions into namespace llvm.

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 09:36:27 PDT 2016


Those locations you mention are in LLDB, and the link you pasted is for a
clang self-host bot that doesn't build LLDB. The build on that bot with
Justin's change was green:
http://lab.llvm.org:8011/builders/clang-x86-windows-msvc2015/builds/757

On Fri, Sep 30, 2016 at 3:04 AM, Stefan van Kessel <van_kessel at freenet.de>
wrote:

> StefanVK added a comment.
>
> This broke the build on windows/msvc (see http://lab.llvm.org:8011/
> builders/clang-x86-windows-msvc2015/builds/766)
>
> Host/windows/Windows.cpp
> Plugins/ObjectFile/PECOFF/WindowsMiniDump.cpp
> Plugins/Process/Windows/MiniDump/ProcessWinMiniDump.cpp
>
> use:
> llvm::UTF16;
> llvm::UTF8;
> llvm::UTF32;
> llvm::ConversionFlags;
> llvm::conversionOK;
> llvm::strictConversion;
>
> But have no using declaration or other means of finding the moved typedefs
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D24996
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160930/99c0d31d/attachment.html>


More information about the llvm-commits mailing list