This revision was automatically updated to reflect the committed changes.
Closed by commit rL284496: [cmake] Update lit search to match the one in LLVM
(authored by mgorny).
Changed prior to commit:
https://reviews.llvm.org/D23745?vs=73192&id=75027#toc
Repository:
rL LLVM
https://reviews.ll
beanz accepted this revision.
beanz added a comment.
This revision is now accepted and ready to land.
Gotcha. LGTM!
https://reviews.llvm.org/D23745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
mgorny added a comment.
In https://reviews.llvm.org/D23745#572517, @beanz wrote:
> Is there a bug you're trying to fix here? It seems to me that the existing
> behavior works correctly. I'm also pretty sure from the comment in
> https://reviews.llvm.org/D23743 that `llvm-lit` shouldn't be in th
beanz added a comment.
Is there a bug you're trying to fix here? It seems to me that the existing
behavior works correctly. I'm also pretty sure from the comment in
https://reviews.llvm.org/D23743 that `llvm-lit` shouldn't be in the list.
https://reviews.llvm.org/D23745
mgorny added a reviewer: beanz.
mgorny added a comment.
A gentle ping here.
https://reviews.llvm.org/D23745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mgorny retitled this revision from "cmake: Adjust llvm-lit search to match the
one in LLVM" to "[cmake] Update lit search to match the one in LLVM".
mgorny updated the summary for this revision.
mgorny updated this revision to Diff 73192.
Herald added a subscriber: beanz.
https://reviews.llvm.org