> On Mar 8, 2016, at 2:46 PM, Ted Woodward <ted.woodw...@codeaurora.org> wrote: > > ted added a comment. > > The change is to guard against the case where candidate_sc.symbol is nullptr. > > candidate_sc.function is only used when load_address != LLDB_INVALID_ADDRESS, > but load_address is set on line 802: > > load_address = candidate_sc.symbol->ResolveCallableAddress(*target); > > so candidate_sc.symbol must be valid. > > The purpose of the function is to get the address of a symbol, so I don't > think we care about candidate_sc.function when candidate_sc.symbol is nullptr.
It's name is "FindInSymbols" but I am pretty sure that's in contradistinction to "FindInRuntimes" not "FindInDebugInformation". The searches that feed this function search both Symbols and Debug Information. I agree with you that the original code worked incorrectly in the case where you had a function from debug information and not from symbols, but your change would need to be reverted to make this work properly (and states an intent that I don't think is correct.) Jim > > > http://reviews.llvm.org/D17860 > > > _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
