diff options
| author | 2018-05-22 16:46:43 +0000 | |
|---|---|---|
| committer | 2018-05-22 16:46:43 +0000 | |
| commit | f31343f1eab771010b3f293366149d2524f30902 (patch) | |
| tree | 5187b2bceee4c7d5347750dc6ba0020172d5ef51 /runtime/thread.cc | |
| parent | e816389b61d17b9a82d8e61426ddfcce20e1238c (diff) | |
| parent | 8268cb677bd92bfbcfec7e803775c29687494e53 (diff) | |
Merge "Remove support for Valgrind in ART."
Diffstat (limited to 'runtime/thread.cc')
| -rw-r--r-- | runtime/thread.cc | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/runtime/thread.cc b/runtime/thread.cc index 81ed722fcc..2275dae5e2 100644 --- a/runtime/thread.cc +++ b/runtime/thread.cc @@ -1115,21 +1115,10 @@ bool Thread::InitStackHwm() { Runtime* runtime = Runtime::Current(); bool implicit_stack_check = !runtime->ExplicitStackOverflowChecks() && !runtime->IsAotCompiler(); - // Valgrind on arm doesn't give the right values here. Do not install the guard page, and - // effectively disable stack overflow checks (we'll get segfaults, potentially) by setting - // stack_begin to 0. - const bool valgrind_on_arm = - (kRuntimeISA == InstructionSet::kArm || kRuntimeISA == InstructionSet::kArm64) && - kMemoryToolIsValgrind && - RUNNING_ON_MEMORY_TOOL != 0; - if (valgrind_on_arm) { - tlsPtr_.stack_begin = nullptr; - } - ResetDefaultStackEnd(); // Install the protected region if we are doing implicit overflow checks. - if (implicit_stack_check && !valgrind_on_arm) { + if (implicit_stack_check) { // The thread might have protected region at the bottom. We need // to install our own region so we need to move the limits // of the stack to make room for it. |