[libcxx-commits] [PATCH] D60525: [pstl] Fix header inclusion order failures

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Apr 11 08:53:50 PDT 2019


ldionne added a comment.

In D60525#1462803 <https://reviews.llvm.org/D60525#1462803>, @MikeDvorskiy wrote:

> Eventually, I tend to that keeping "cout" code is redundant , just "return 0" is enough.
>  We will try to re-config our test system to accept just returning zero;


That would be great! Another nice side effect of this is that we could stop using `<iostreams>` everywhere, which is both not really in line with the rest of the libc++ test suite, and also extremely costly in terms of compilation time. I'm sure you've noticed that the tests were not exactly fast to compile :-).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60525





More information about the libcxx-commits mailing list