[PATCH] D58940: [Sanitizer] Only unwind when it is safe to do so

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 4 20:20:05 PST 2019


vitalybuka added inline comments.


================
Comment at: compiler-rt/lib/asan/asan_stack.cc:37
 
   AsanThread *t = GetCurrentThread();
+  if (!t || t->isUnwinding()) return;
----------------
We can't assume that there is no issues here so quickly after commit.
I will take time to get Android.

I believe I've made mistake proposing the current code D58861



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D58940





More information about the llvm-commits mailing list