diff options
author | Aiden Grossman <aidengrossman@google.com> | 2025-09-19 16:36:37 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-09-19 16:36:37 -0700 |
commit | a38794ff3d47588cb226881eb048cb2333962ab9 (patch) | |
tree | ffa738da6db2b5498a1d1af7977d58fe9d351fc0 /clang/lib/Sema/SemaModule.cpp | |
parent | 9542d0a0c661be92db950514b5dc9c5ea6d953af (diff) | |
download | llvm-a38794ff3d47588cb226881eb048cb2333962ab9.zip llvm-a38794ff3d47588cb226881eb048cb2333962ab9.tar.gz llvm-a38794ff3d47588cb226881eb048cb2333962ab9.tar.bz2 |
[Clang] Rewrite tests using subshells to set env variables
Now that we have the %readfile substitution, we can rewrite these tests
that were using env variable subshells to write the output of the
command into a file and then load it where it is needed using readfile.
This does involve one invocation of bash so that we are using the system
env binary, which does support redirection into a tool like grep. We
already do this in one LLVM test. I'm not happy about needing that, but
the more principled way to solve it involves reworking how in-process
builtins work within lit. That is something we want to do eventually,
but not something that I think should block this patch.
Reviewers: cmtice, petrhosek, ilovepi
Reviewed By: cmtice, ilovepi
Pull Request: https://github.com/llvm/llvm-project/pull/158446
Diffstat (limited to 'clang/lib/Sema/SemaModule.cpp')
0 files changed, 0 insertions, 0 deletions