[Lldb-commits] [PATCH] D56904: [NativePDB] Process virtual bases in the correct order

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Tue Feb 5 08:38:26 PST 2019


Yes, clang tries to find the visual studio installation, because that is
how it can find headers and libs.  Can you make a separate patch with the
changes to build.py and upload that?

On Tue, Feb 5, 2019 at 2:34 AM Aleksandr Urakov via Phabricator <
reviews at reviews.llvm.org> wrote:

> aleksandr.urakov added a comment.
>
> Thanks for the reply! It seems that after your hint I've figured it out.
> It's the environment variable `VCINSTALLDIR`. I've added it to the
> `defaultenv` list of the `_get_visual_studio_environment` function and the
> test passes now. What do you think about this solution?
>
> And do I understand right, it means, that clang-cl needs MSVC installation
> to work with these types?
>
>
> Repository:
>   rLLDB LLDB
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D56904/new/
>
> https://reviews.llvm.org/D56904
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190205/5ae54c63/attachment.html>


More information about the lldb-commits mailing list