[llvm] [NFC][bolt][test] Change '|&' to '2>&1 |' for lit internal shell support (PR #102402)
Vladislav Khmelevsky via llvm-commits
llvm-commits at lists.llvm.org
Sat Aug 10 00:54:48 PDT 2024
yota9 wrote:
> > Hi. I'm not too deep in lit internal shell thematics, but it won't prevent on creating new tests with 2>&1 syntax. Is there any reason that 2>&1 is not supported by lit shell? Maybe it should be fixed there?
>
> `2>&1` is supported. This change removes the bashism `|&`, and replaces it w/ `2>&1`, unless I've misunderstood.
Haha, it's better to just go sleep at nights, I meant the opposite of course :)
> Also, this is related to this RFC: https://discourse.llvm.org/t/rfc-enabling-the-lit-internal-shell-by-default/80179
Thanks for the link!
https://github.com/llvm/llvm-project/pull/102402
More information about the llvm-commits
mailing list