[Lldb-commits] [PATCH] D48960: Use an unwinder to get register contexts of frames other than zeroth under Windows
Adrian McCarthy via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jul 9 16:19:00 PDT 2018
amccarth added inline comments.
================
Comment at: source/Plugins/Process/Windows/Common/TargetThreadWindows.cpp:52
if (!m_reg_context_sp)
- m_reg_context_sp = CreateRegisterContextForFrameIndex(0);
+ m_reg_context_sp = CreateRegisterContextForZerothFrame();
----------------
This bit seems redundant given that CreateRegisterContextForFrameIndex now checks the special case of 0. It's not a big deal, but I could see someone getting confused while debugging things wondering by CreateRegisterContextForFrameIndex is sometimes called for frame 0 and sometimes not.
https://reviews.llvm.org/D48960
More information about the lldb-commits
mailing list