rsmith added a comment.

In D77184#1961214 <https://reviews.llvm.org/D77184#1961214>, @rsmith wrote:

> In D77184#1961208 <https://reviews.llvm.org/D77184#1961208>, @rsmith wrote:
>
> > This has broken my ability to run the `check-clang` target on Linux. There 
> > are symlinks in the path from which I run my builds, and this patch is 
> > computing incorrect relative paths in that situation.
>
>
> This patch appears to fix the problem:


Turns out that's only a partial fix (it fixes `ninja clang-check` but not the 
general problem). Running `lit` from inside the source directory is still 
broken. Eg, this used to work:

  clang/test$ $BUILDDIR/bin/llvm-lit SemaCXX/constant-expression.cpp

... and doesn't work any more because the relative paths in `lit.site.cfg.py` 
don't seem to resolve to the right places. Please can you fix or revert?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77184/new/

https://reviews.llvm.org/D77184



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to