[PATCH] D65697: [lit] Fix internal env calling env
Paul Robinson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 21 15:11:52 PDT 2019
probinson added a comment.
In D65697#1640136 <https://reviews.llvm.org/D65697#1640136>, @jdenny wrote:
> In D65697#1640109 <https://reviews.llvm.org/D65697#1640109>, @rnk wrote:
>
> > Most LLVM test suites are configured to use the internal shell on Windows. I think what's happening is that users happen to have an external copy of `diff` that gets called when it appears in a pipeline, or they are prefixed by `not`. The external `diff` is called, and things "work out" and we never noticed.
>
>
> Are these patches going to break those tests on Windows? We now see `diff` anywhere in a pipeline, and we reject its appearance in a pipeline.
`grep -r -l 'RUN:.* diff ' llvm/test clang/test` gets 240 hits. Searching on `| diff` gets 41, mostly in clang/test/Analysis. So not a super common pattern, but if it's straightforward to build a Python 'diff' that handles pipes, that would be less invasive, and piping does have a minor speed advantage.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65697/new/
https://reviews.llvm.org/D65697
More information about the llvm-commits
mailing list