[Lldb-commits] [PATCH] D76806: Remove m_last_file_sp from SourceManager

[Jim Ingham via Phabricator via lldb-commits]

jingham accepted this revision.
jingham added a comment.

There was one missing use of GetLastFile (as opposed to doing it by hand.)

Getting the default file is not used in any performance critical way that I'm aware of.  It's mostly used for "break set -l" with no "-f" and list with no arguments and commands of that sort.  So the extra lookup to get the FileSP from the source manager instead of caching it directly should not cause problems.  And if you asked not to cache, you've already decided to pay the cost for that.

LGTM with that trivial fix.



================
Comment at: lldb/source/Core/SourceManager.cpp:175-176
 
-  if (m_last_file_sp.get()) {
+  FileSP last_file_sp(GetFile(m_last_file_spec));
+  if (last_file_sp.get()) {
     const uint32_t end_line = start_line + count - 1;
----------------
labath wrote:
> `if(FileSP last_file_sp = GetLastFile())`
Doesn't look like this got converted to GetLastFile.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76806/new/

https://reviews.llvm.org/D76806