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

Reply via email to