diff options
author | 2021-05-26 17:33:54 +0000 | |
---|---|---|
committer | 2021-05-27 19:28:16 +0000 | |
commit | e8efdaa09b7ff36d5a986f0320f29ac4a7563896 (patch) | |
tree | 5066cecab43006a7bf23f793aa911f74559c8017 /runtime/interpreter/interpreter_common.cc | |
parent | ae553e0e1b26d95006aa27df4675104bfb55d49d (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.cc | 2 |
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. |