Send stdout to dev/null since the preprocessor output isn't relevant to the test and is tons of noise when the test fails.
Details
Details
- Reviewers
ldionne Mordante philnik - Group Reviewers
Restricted Project - Commits
- rG63959a18e74d: [libc++] test/libcxx/transitive_includes.sh.cpp dumps loads of unhelpful…
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
libcxx/test/libcxx/transitive_includes.sh.cpp | ||
---|---|---|
60 | This is the only actual changed line, the rest of them were done by libcxx-generate-files. We could also do >&- to close stdout, but the internet suggests that's a bit spotty. Although it does work for at least clang, if people prefer that method. |
Comment Actions
I don't think we do suddenly get more output, I think it's always been this way. You just don't see stdout unless the test fails.
Comment Actions
I don't recall seeing that when I worked on patches to improve this test. But at least it's fixed now :-)
This is the only actual changed line, the rest of them were done by libcxx-generate-files. We could also do >&- to close stdout, but the internet suggests that's a bit spotty. Although it does work for at least clang, if people prefer that method.