diff options
author | Aiden Grossman <aidengrossman@google.com> | 2025-09-09 13:37:13 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-09-09 13:37:13 -0700 |
commit | 3e790bd03a970e05f1717d91f870cbdfb69a0b78 (patch) | |
tree | 41eced657354716e48b098245e7afe918eb8196e /llvm/lib/FileCheck/FileCheck.cpp | |
parent | aa9af2a3ec4c986fc8a7103a2fbbcdc4ccc079e4 (diff) | |
download | llvm-3e790bd03a970e05f1717d91f870cbdfb69a0b78.zip llvm-3e790bd03a970e05f1717d91f870cbdfb69a0b78.tar.gz llvm-3e790bd03a970e05f1717d91f870cbdfb69a0b78.tar.bz2 |
[not] Update disable-symbolization.test to work with internal shell
This test is a special case as it executes two commands that are special
cased in the internal shell implementation. env runs entirely inside the
internal shell whereas not is handled specially, but still executed
externally. The internal shell does reorder execution of these though,
putting env commands before not which means we do not pick up
environment variables set by not.
These complications make it easier to just ensure that we invoke the
actual env binary (by calling it through bash) rather than using the
internal shell implementation.
Fixes #106627 by fixing the test, but without fixing the redirection
issue given the complexity does not seem justified.
Reviewers: ilovepi, MaskRay, petrhosek
Reviewed By: MaskRay
Pull Request: https://github.com/llvm/llvm-project/pull/157236
Diffstat (limited to 'llvm/lib/FileCheck/FileCheck.cpp')
0 files changed, 0 insertions, 0 deletions