simark added a comment.

In https://reviews.llvm.org/D48903#1152485, @ilya-biryukov wrote:

> I usually run `ninja check-clang check-clang-tools` for clang changes. Have 
> never used `clang-test`, not sure what it does.


I think `clang-test` is an alias for `check-clang`.

> I ran it with this change, found a few failures from clang-move:
> 
>   Failing Tests (5):
>       Extra Tools Unit Tests :: 
> clang-move/./ClangMoveTests/ClangMove.AddDependentNewHeader
>       Extra Tools Unit Tests :: 
> clang-move/./ClangMoveTests/ClangMove.AddDependentOldHeader
>       Extra Tools Unit Tests :: 
> clang-move/./ClangMoveTests/ClangMove.DontMoveAll
>       Extra Tools Unit Tests :: 
> clang-move/./ClangMoveTests/ClangMove.MoveHeaderAndCC
>       Extra Tools Unit Tests :: 
> clang-move/./ClangMoveTests/ClangMove.MoveHeaderOnly

Doh, I'll run `check-clang-tools` too.


Repository:
  rC Clang

https://reviews.llvm.org/D48903



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

Reply via email to