diff options
author | Mircea Trofin <mtrofin@google.com> | 2023-02-05 17:16:46 -0800 |
---|---|---|
committer | Mircea Trofin <mtrofin@google.com> | 2023-02-06 13:57:16 -0800 |
commit | 1b80ccba0f9bb86eabeeb0c32876eae2088eafb8 (patch) | |
tree | a01369b8253d6aae0d9342925637e1cdb0b5b16b /lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPythonImpl.h | |
parent | cb5e48d1c2c4774ed9f17ff89412f1291b640172 (diff) | |
download | llvm-1b80ccba0f9bb86eabeeb0c32876eae2088eafb8.zip llvm-1b80ccba0f9bb86eabeeb0c32876eae2088eafb8.tar.gz llvm-1b80ccba0f9bb86eabeeb0c32876eae2088eafb8.tar.bz2 |
[mlgo][regalloc] Handle training case when no regalloc happens.
There's an early-exit case for regalloc when we don't even get a chance
to ask for an advisor (priority or eviction), and switch the context.
Then, when we want to log the reward for that function (==the one with
the early exit case), we hit the error case where the function's name
doesn't match the last-seen context.
There are a few possible fixes, one would be to just switch context when
output-ing the reward, which would be correct. This patch opts for the
alternative where we check any loging happened in the first place - just
to re-validate that no function would have been regaloc-ed without first
log-ing its reward.
Differential Revision: https://reviews.llvm.org/D143359
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPythonImpl.h')
0 files changed, 0 insertions, 0 deletions