summaryrefslogtreecommitdiff
path: root/runtime/interpreter/interpreter_common.cc
diff options
context:
space:
mode:
author Christian Wailes <chriswailes@google.com> 2021-05-26 17:33:54 +0000
committer Treehugger Robot <treehugger-gerrit@google.com> 2021-05-27 19:28:16 +0000
commite8efdaa09b7ff36d5a986f0320f29ac4a7563896 (patch)
tree5066cecab43006a7bf23f793aa911f74559c8017 /runtime/interpreter/interpreter_common.cc
parentae553e0e1b26d95006aa27df4675104bfb55d49d (diff)
Revert "Remove stack protector from some interpreter functions"
A performance regression was noticed in go/lem banchmarks for ART. This is likely caused by the no_stack_protector attribute preventing inlining into hot functions. Reason for revert: performance regression Reverted Changes: Ie0c81f97f:Remove stack protector from some interpreter funct... Iaf565a374:Reset stack guard after fork from Zygote Bug: 168258494 Change-Id: I018d58759c3a5bb624ea8e7fe09b6b49c9b19b6a
Diffstat (limited to 'runtime/interpreter/interpreter_common.cc')
-rw-r--r--runtime/interpreter/interpreter_common.cc2
1 files changed, 0 insertions, 2 deletions
diff --git a/runtime/interpreter/interpreter_common.cc b/runtime/interpreter/interpreter_common.cc
index 4ae7fb1c20..4e88eda162 100644
--- a/runtime/interpreter/interpreter_common.cc
+++ b/runtime/interpreter/interpreter_common.cc
@@ -250,7 +250,6 @@ ALWAYS_INLINE void CopyRegisters(ShadowFrame& caller_frame,
// END DECLARATIONS.
-NO_STACK_PROTECTOR
void ArtInterpreterToCompiledCodeBridge(Thread* self,
ArtMethod* caller,
ShadowFrame* shadow_frame,
@@ -1413,7 +1412,6 @@ static inline bool DoCallCommon(ArtMethod* called_method,
}
template<bool is_range, bool do_assignability_check>
-NO_STACK_PROTECTOR
bool DoCall(ArtMethod* called_method, Thread* self, ShadowFrame& shadow_frame,
const Instruction* inst, uint16_t inst_data, JValue* result) {
// Argument word count.