[PATCH] D38680: [libunwind] Fix handling of DW_CFA_GNU_args_size
Reid Kleckner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 11 14:53:44 PDT 2017
rnk added inline comments.
================
Comment at: src/libunwind.cpp:188
+ co->getInfo(&info);
+ pint_t orgArgSize = (pint_t)info.gp;
+ uint64_t orgFuncStart = info.start_ip;
----------------
I think it makes sense to have this here: the contract is that if the personality sets the IP when the cursor pointed to a PC with a non-zero arg size, we should adjust SP for the personality.
However, it's not clear to me that we don't need the same adjustment when stepping across frames that use arg size without a frame pointer.
https://reviews.llvm.org/D38680
More information about the cfe-commits
mailing list