[lld] [lld] Update tests redirecting multiple commands to FileCheck (PR #156526)
Aiden Grossman via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 3 09:28:15 PDT 2025
boomanaiden154 wrote:
> It's a shame that the internal shell doesn't support (llvm-objdump --section-headers %t/test; dsymutil -s %t/test) | FileCheck %s. The split form is more verbose and less readable. env XXX=1 changes look good.
Yeah, it's definitely not as nice. Given there are very few tests that use this construction and rewriting them to do without isn't that much worse, I think it's hard to justify the complexity in adding support to lit's internal shell to support this construction. It might be good to revisit that at some point though.
https://github.com/llvm/llvm-project/pull/156526
More information about the llvm-commits
mailing list