[PATCH] [PowerPC]Adapt fast stack unwinding to work for Power.

hfinkel at anl.gov hfinkel at anl.gov
Fri Apr 24 18:52:55 PDT 2015


Can you please post this patch with full context; see: http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface


================
Comment at: /home/seurer/llvm/llvm-test/projects/compiler-rt/lib/sanitizer_common/sanitizer_stacktrace.cc:93
@@ +92,3 @@
+  if (FAST_UNWIND_MAY_NEED_TO_POP_FRAMES) {
+    // We possibly need to pop a few frames so that pc is on top.
+    PopStackFramesToPc(pc);
----------------
Please explain why.

http://reviews.llvm.org/D9259

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list