diff options
author | Michael Buch <michaelbuch12@gmail.com> | 2024-04-11 20:29:18 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-04-11 20:29:18 +0200 |
commit | fc52ee336b394d84110184d625cda1d4f84d8098 (patch) | |
tree | aba42afda2880d146aa8794297748caf32b6f6d7 /clang/lib/CodeGen/CodeGenModule.cpp | |
parent | b2ea38f9fc2381e7c04e610f6f8ed6786c2da38e (diff) | |
download | llvm-fc52ee336b394d84110184d625cda1d4f84d8098.zip llvm-fc52ee336b394d84110184d625cda1d4f84d8098.tar.gz llvm-fc52ee336b394d84110184d625cda1d4f84d8098.tar.bz2 |
[lldb][ClangUserExpression][NFCI] Pass the most specific ExecutionContextScope possible into ClangExpressionParser (#87657)
The `ClangExpressionParser` takes an `ExecutionContextScope` which it
uses to query the `Process`/`Target`/`StackFrame` to set various
compiler options in preparation for parsing an expression.
However, `TryParse` constructs the parser with a `Process` or `Target`,
never a `StackFrame`. So when the parser tries to retrieve the current
`StackFrame` from the `exe_scope`, it doesn't succeed. In future patches
we want to query the `StackFrame` from within the
`ClangExpressionParser` constructor.
This patch simplifies `TryParse`, by removing the redundant `exe_scope`
parameter, and instead uses the `exe_ctx` to derive the most fitting
`exe_scope` to pass into `ClangExpressionParser`.
Not entirely sure how to test this. This patch is a prerequisite to get
subsequent patches that set `LangOpts` based on the current `StackFrame`
to work.
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.cpp')
0 files changed, 0 insertions, 0 deletions