boomanaiden154 wrote: > Should we update the lit implementation handling for not/env to avoid > reordering those two commands when combined?
We theoretically can, but it's not that simple. The test runner already explicitly reorders them due to how `not` needs to be run. I'm not sure of exact test cases where the following is relevant, but it seems like it would be difficult to tease apart, and was not worth it for this test based on my assessment: https://github.com/llvm/llvm-project/blob/d2646cac857294604afaae67f165323b0af35430/llvm/utils/lit/lit/TestRunner.py#L851 https://github.com/llvm/llvm-project/pull/157236 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
