[Lldb-commits] [lldb] [lldb] Implement (SB)Function::GetInstructions for discontinuous functions (PR #122933)

[Pavel Labath via lldb-commits]

================
@@ -165,10 +169,14 @@ bool SBInstructionList::GetDescription(Stream &sref) {
               addr, eSymbolContextEverything, sc);
         }
 
+        if (next_addr && addr != next_addr)
----------------
labath wrote:

It doesn't *need* to be because `std::optional<T>` defines an operator== for comparison with plain `T`s (which treats an empty optional as different from all values of `T`), but yes, it would be better to compare the `T` directly since that's not the behavior I want (I basically want to treat the empty optional as *equal* to any address).

https://github.com/llvm/llvm-project/pull/122933