jingham marked 6 inline comments as done. jingham added inline comments.
================ Comment at: lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp:1012-1015 + if (log) { + LLDB_LOG(log, "Resolving call for class - {0} and selector - {1}", + isa_addr, sel_addr); + } ---------------- JDevlieghere wrote: > The `if (log)` check is redundant. It's the purpose of the macro to expand to > that that. Same below. Old habits die hard... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123421/new/ https://reviews.llvm.org/D123421 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits