[libcxx-commits] [PATCH] D128998: [libunwind] Handle G in personality string

[Evgenii Stepanov via Phabricator via libcxx-commits]

eugenis added inline comments.


================
Comment at: libunwind/src/DwarfInstructions.hpp:221
+        p &= ~15ULL;
+        for (; p < cfa; p += 16) {
+          __asm__ __volatile__(
----------------
Is this right? I'd expect to untag from current frame SP to previous frame SP, not to CFA (which could be at arbitrary offset in the frame AFAIK).



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128998/new/

https://reviews.llvm.org/D128998