diff options
| author | Matthias Braun <matze@braunis.de> | 2016-01-26 00:43:50 +0000 |
|---|---|---|
| committer | Matthias Braun <matze@braunis.de> | 2016-01-26 00:43:50 +0000 |
| commit | 242b8bb58dee9556e310e7f57f654dc64c6fad7b (patch) | |
| tree | 3fa23c200150e85b38068ee04e911106943c8d11 /lldb/packages/Python/lldbsuite/test/python_api/exprpath_synthetic | |
| parent | 75452734e47a0d5bcf108465b1930ca500cccfa2 (diff) | |
| download | bcm5719-llvm-242b8bb58dee9556e310e7f57f654dc64c6fad7b.tar.gz bcm5719-llvm-242b8bb58dee9556e310e7f57f654dc64c6fad7b.zip | |
LiveIntervalAnalysis: Cleanup handleMove{Down|Up}() functions, NFC
These two functions are hard to reason about. This commit makes the code
more comprehensible:
- Use four distinct variables (OldIdxIn, OldIdxOut, NewIdxIn, NewIdxOut)
with a fixed value instead of a changing iterator I that points to
different things during the function.
- Remove the early explanation before the function in favor of more
detailed comments inside the function. Should have more/clearer comments now
stating which conditions are tested and which invariants hold at
different points in the functions.
The behaviour of the code was not changed.
I hope that this will make it easier to review the changes in
http://reviews.llvm.org/D9067 which I will adapt next.
Differential Revision: http://reviews.llvm.org/D16379
llvm-svn: 258756
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api/exprpath_synthetic')
0 files changed, 0 insertions, 0 deletions
