[PATCH] D60525: [pstl] Fix header inclusion order failures
Louis Dionne via Phabricator via llvm-commits
llvm-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 llvm-commits
mailing list