[PATCH] [Commented On] D24996: Move UTF functions into namespace llvm.
Stefan van Kessel via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 30 10:47:08 PDT 2016
You are right, I accidentally copied the link to another build bot.
Sorry for the confusion.
This is a build which appears to have failed in relation to rL282822:
http://lab.llvm.org:8011/builders/lldb-windows7-android/builds/8692
Anyway it's fixed now. Thank you all.
On 2016-09-30 18:36, Reid Kleckner via llvm-commits wrote:
> 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 <mailto: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
> <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 <https://reviews.llvm.org/D24996>
>
>
>
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
More information about the llvm-commits
mailing list