[PATCH] D54187: Add debuginfo-tests that use cdb on Windows

Adrian Prantl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 7 09:17:58 PST 2018


aprantl added a comment.

In https://reviews.llvm.org/D54187#1290317, @zturner wrote:

> In https://reviews.llvm.org/D54187#1290297, @aprantl wrote:
>
> > In https://reviews.llvm.org/D54187#1290293, @zturner wrote:
> >
> > > Especially since as far as I can tell, nobody has even run debuginfo-tests since late August, because it was actually broken by r341135 on August 30 (fixed in r346060 yesterday)
> >
> >
> > Can you please refrain from making such general statements? They distract from the discussion.
> >
> > http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-expensive/ runs the debuginfo-tests continuously. There is a configuration issue that prevents it from running the ASAN subset of the tests at the moment.
>
>
> The issue introduced by r341135 doesn't seem related to ASAN unless I'm misunderstanding the issue.  Were debuginfo-tests passing on that bot even before r346060 landed?


r341135 is a fix to `lit.site.cfg.py.in` my best guess is that because clang-stage1-cmake-RA-expensive runs the debuginfo-tests inside the llvm/tools/clang/test/debuginfo-tests subdirectory that doesn't get used in that configuration?


https://reviews.llvm.org/D54187





More information about the cfe-commits mailing list