diff options
author | Pavel Labath <labath@google.com> | 2017-12-14 14:56:45 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2017-12-14 14:56:45 +0000 |
commit | f1208e7aefc1c36620370e058af932b7b9bf3bb8 (patch) | |
tree | 623b098685e75d3dc80731c3cf93b5694310a0cb /clang/lib/Format/ContinuationIndenter.cpp | |
parent | 8630d3871d7d3de07a2615835b3b6e1357aa564a (diff) | |
download | bcm5719-llvm-f1208e7aefc1c36620370e058af932b7b9bf3bb8.tar.gz bcm5719-llvm-f1208e7aefc1c36620370e058af932b7b9bf3bb8.zip |
ObjectFile: remove ReadSectionData/MemoryMapSectionData mutual recursion
Summary:
These two functions were calling each other, while handling different
branches of the if(IsInMemory()). This had a reason at some point in the
past, but right now it's just confusing.
I resolve this by removing the MemoryMapSectionData function and
inlining the !IsInMemory branch into ReadSectionData. There isn't
anything mmap-related in this function anyway, as the decision whether
to mmap is handled at a higher level.
This is a preparatory step to make ObjectFileELF be able to decompress
compressed sections (I want to make sure that all calls reading section
data are routed through a single piece of code).
Reviewers: clayborg
Subscribers: emaste, JDevlieghere, lldb-commits
Differential Revision: https://reviews.llvm.org/D41169
llvm-svn: 320705
Diffstat (limited to 'clang/lib/Format/ContinuationIndenter.cpp')
0 files changed, 0 insertions, 0 deletions