summaryrefslogtreecommitdiff
path: root/runtime/thread.cc
diff options
context:
space:
mode:
author Roland Levillain <rpl@google.com> 2018-05-22 16:46:43 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2018-05-22 16:46:43 +0000
commitf31343f1eab771010b3f293366149d2524f30902 (patch)
tree5187b2bceee4c7d5347750dc6ba0020172d5ef51 /runtime/thread.cc
parente816389b61d17b9a82d8e61426ddfcce20e1238c (diff)
parent8268cb677bd92bfbcfec7e803775c29687494e53 (diff)
Merge "Remove support for Valgrind in ART."
Diffstat (limited to 'runtime/thread.cc')
-rw-r--r--runtime/thread.cc13
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.