[lldb/API] Add SBCompileUnit::GetIndexForLineEntry method to SB API

[mib]

Summary

This patch adds a new GetIndexForLineEntry method to the SBCompileUnit
class. As the name suggests, given an SBLineEntry object, this will
return the line entry index within a specific compile unit.

This method can take a exact boolean that will make sure that the
provided line entry matches perfectly another line entry in the compile unit.

rdar://47450887

Signed-off-by: Med Ismail Bennani <[e-mail]>

[jingham]

This needs a doctoring for the API and some test.

lldb/source/API/SBCompileUnit.cpp

86

llvm would prefer an early return here.

94

Why do you have to do this Compare? You already passed exact to FindLineEntry, does FindLineEntry really return line entries that fail your Compare test when exact is passed in as true? That doesn't seem right.

[mib]

lldb/source/API/SBCompileUnit.cpp

94

exact could be false, in which case the line entry index returned by FindLineEntry will be different from the one provided by the user.

But if the user really wants the exact line entry index, we need to compare the returned line entry and the one provided by the user.

[mib]

Addressed @jingham comments:

• Use early returns
• Remove LineEntry::Compare call since it's already handled by CompileUnit::FindLineEntry
• Add docstring

[mib]

I forgot to add the test in the previous diffs --'

[mib]

Address @jingham feedbacks

[jingham]

LGTM

[JDevlieghere]

lldb/include/lldb/API/SBCompileUnit.h

37

How's this different from FindLineEntryIndex? In other words why not make this an FindLineEntryIndex overload? It would be good to document this here (too).

[mib]

Actually, landed this yesterday in a6926d576131c9ad849fef6f1d43134caab5025e

lldb/include/lldb/API/SBCompileUnit.h

37

We already have a line entry but we just want to get the index in the compile unit, so it made sense to have a separate method for this.

[JDevlieghere]

lldb/include/lldb/API/SBCompileUnit.h

37

I was referring to the name. It seems weird to have a few overloads of FindLineEntryIndex and then a totally different function GetIndexForLineEntry. Are they conceptually different enough to have different names? Don't they both return the line entry index?

lldb/bindings/interface/SBCompileUnit.i

public:
 
     uint32_t
     GetNumLineEntries () const;
 
     lldb::SBLineEntry
     GetLineEntryAtIndex (uint32_t idx) const;
 
     uint32_t
+    GetIndexForLineEntry (const lldb::SBLineEntry &line_entry, bool exact = false) const;
+
+    uint32_t
     FindLineEntryIndex (uint32_t start_idx,
                         uint32_t line,
                         lldb::SBFileSpec *inline_file_spec) const;
 
     uint32_t
     FindLineEntryIndex (uint32_t start_idx,
                         uint32_t line,
                         lldb::SBFileSpec *inline_file_spec,

lldb/include/lldb/API/SBCompileUnit.h

public:
   bool IsValid() const;
 
   lldb::SBFileSpec GetFileSpec() const;
 
   uint32_t GetNumLineEntries() const;
 
   lldb::SBLineEntry GetLineEntryAtIndex(uint32_t idx) const;
 
+  uint32_t GetIndexForLineEntry(const lldb::SBLineEntry &line_entry,
+                                bool exact = false) const;
+
   uint32_t FindLineEntryIndex(uint32_t start_idx, uint32_t line,
                               lldb::SBFileSpec *inline_file_spec) const;
 
   uint32_t FindLineEntryIndex(uint32_t start_idx, uint32_t line,
                               lldb::SBFileSpec *inline_file_spec,
                               bool exact) const;
 
   SBFileSpec GetSupportFileAtIndex(uint32_t idx) const;

lldb/source/API/SBCompileUnit.cpp

if (line_table) {
       if (line_table->GetLineEntryAtIndex(idx, line_entry))
         sb_line_entry.SetLineEntry(line_entry);
     }
   }
 
   return sb_line_entry;
 }
 
+uint32_t
+SBCompileUnit::GetIndexForLineEntry(const lldb::SBLineEntry &line_entry,
+                                    bool exact) const {
+  LLDB_INSTRUMENT_VA(this, line_entry, exact);
+
+  uint32_t index = UINT32_MAX;
+  if (m_opaque_ptr && line_entry.IsValid()) {
+
+    LineEntry found_line_entry;
+
+    uint32_t found_index = m_opaque_ptr->FindLineEntry(
+        0, line_entry.GetLine(), line_entry.GetFileSpec().get(), exact,
+        &found_line_entry);
+
+    if (!exact ||
+        (exact && !LineEntry::Compare(line_entry.ref(), found_line_entry)))
+      index = found_index;
+  }
+
+  return index;
+}
+
 uint32_t SBCompileUnit::FindLineEntryIndex(uint32_t start_idx, uint32_t line,
                                            SBFileSpec *inline_file_spec) const {
   LLDB_INSTRUMENT_VA(this, start_idx, line, inline_file_spec);
 
   const bool exact = true;
   return FindLineEntryIndex(start_idx, line, inline_file_spec, exact);
 }