forked from OSchip/llvm-project
Revert "[LSAN] Fix test swapcontext.cc on MIPS"
This reverts commit r266716, as it breaks the self-hosting on Thumb2 buildbot. llvm-svn: 267158
This commit is contained in:
parent
9fe26e6dc9
commit
e48f7e6f2d
|
@ -48,10 +48,7 @@ void GetStackTraceWithPcBpAndContext(BufferedStackTrace *stack, uptr max_depth,
|
|||
uptr stack_top = t->stack_top();
|
||||
uptr stack_bottom = t->stack_bottom();
|
||||
ScopedUnwinding unwind_scope(t);
|
||||
if (IsValidFrame(bp, stack_top, stack_bottom)) {
|
||||
stack->Unwind(max_depth, pc, bp, context, stack_top, stack_bottom,
|
||||
fast);
|
||||
}
|
||||
stack->Unwind(max_depth, pc, bp, context, stack_top, stack_bottom, fast);
|
||||
} else if (!t && !fast) {
|
||||
/* If GetCurrentThread() has failed, try to do slow unwind anyways. */
|
||||
stack->Unwind(max_depth, pc, bp, context, 0, 0, false);
|
||||
|
|
|
@ -20,14 +20,12 @@
|
|||
{ \
|
||||
uptr stack_top = 0, stack_bottom = 0; \
|
||||
ThreadContext *t; \
|
||||
if ((t = CurrentThreadContext())) { \
|
||||
if (fast && (t = CurrentThreadContext())) { \
|
||||
stack_top = t->stack_end(); \
|
||||
stack_bottom = t->stack_begin(); \
|
||||
} \
|
||||
if (IsValidFrame(GET_CURRENT_FRAME(), stack_top, stack_bottom)) { \
|
||||
stack.Unwind(max_size, StackTrace::GetCurrentPc(), GET_CURRENT_FRAME(), \
|
||||
/* context */ 0, stack_top, stack_bottom, fast); \
|
||||
} \
|
||||
stack.Unwind(max_size, StackTrace::GetCurrentPc(), GET_CURRENT_FRAME(), \
|
||||
/* context */ 0, stack_top, stack_bottom, fast); \
|
||||
}
|
||||
|
||||
#define GET_STACK_TRACE_FATAL \
|
||||
|
|
|
@ -40,6 +40,11 @@ void BufferedStackTrace::Init(const uptr *pcs, uptr cnt, uptr extra_top_pc) {
|
|||
top_frame_bp = 0;
|
||||
}
|
||||
|
||||
// Check if given pointer points into allocated stack area.
|
||||
static inline bool IsValidFrame(uptr frame, uptr stack_top, uptr stack_bottom) {
|
||||
return frame > stack_bottom && frame < stack_top - 2 * sizeof (uhwptr);
|
||||
}
|
||||
|
||||
// In GCC on ARM bp points to saved lr, not fp, so we should check the next
|
||||
// cell in stack to be a saved frame pointer. GetCanonicFrame returns the
|
||||
// pointer to saved frame pointer in any case.
|
||||
|
|
|
@ -110,11 +110,6 @@ struct BufferedStackTrace : public StackTrace {
|
|||
void operator=(const BufferedStackTrace &);
|
||||
};
|
||||
|
||||
// Check if given pointer points into allocated stack area.
|
||||
static inline bool IsValidFrame(uptr frame, uptr stack_top, uptr stack_bottom) {
|
||||
return frame > stack_bottom && frame < stack_top - 2 * sizeof (uhwptr);
|
||||
}
|
||||
|
||||
} // namespace __sanitizer
|
||||
|
||||
// Use this macro if you want to print stack trace with the caller
|
||||
|
|
Loading…
Reference in New Issue