[Lldb-commits] [PATCH] D17860: Fix "ninja check-lldb" crash in IRExecutionUnit.cpp
Sean Callanan via lldb-commits
lldb-commits at lists.llvm.org
Tue Mar 8 10:36:59 PST 2016
spyffe accepted this revision.
spyffe added a comment.
Looks good to me. Thanks for the fix!
================
Comment at: source/Expression/IRExecutionUnit.cpp:802
@@ -801,3 +801,3 @@
- load_address = candidate_sc.symbol->ResolveCallableAddress(*target);
+ load_address = candidate_sc.symbol->ResolveCallableAddress(*target);
----------------
Ah yes, I see what's being fixed here. This looks fine. I like Adrian's recommended early-out policy but this isn't a hard-and-fast rule. It seems like doing the logic the way you're doing it is less invasive, so I'm fine with doing it this way for now.
http://reviews.llvm.org/D17860
More information about the lldb-commits
mailing list