boomanaiden154 wrote:
> I don't know if I understand what you're going for in this patch. How is this
> an improvement over just keeping REQUIRES: shell? The REQUIRES: bit is
> what's going to stop the test from running in the wrong environment. That
> seems like the right tradeoff.
If we do
https://github.com/boomanaiden154 created
https://github.com/llvm/llvm-project/pull/157232
This test needs to use subshells so that the file descriptor offset gets
moved before llvm-mc starts writing output. I cannot get this to trigger
the assertion (after reverting the code fixes that fixed th
ilovepi wrote:
I don't know if I understand what you're going for in this patch. How is this
an improvement over just keeping `REQUIRES: shell`? The `REQUIRES: ` bit is
what's going to stop the test from running in the wrong environment. That seems
like the right tradeoff.
https://github.c
