You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2020/04/27 23:27:03 UTC
[groovy] branch GROOVY_3_0_X updated: Eliminate redundant checks
and trying to cache spread calls
This is an automated email from the ASF dual-hosted git repository.
sunlan pushed a commit to branch GROOVY_3_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
The following commit(s) were added to refs/heads/GROOVY_3_0_X by this push:
new dccaea3 Eliminate redundant checks and trying to cache spread calls
dccaea3 is described below
commit dccaea3068b3e68f5f5b89018b20b99a123904a9
Author: Daniel Sun <su...@apache.org>
AuthorDate: Tue Apr 28 07:25:48 2020 +0800
Eliminate redundant checks and trying to cache spread calls
spread calls won't be cacheable
(cherry picked from commit e3c39cb959a4519f60b013f0bba3701fe921c2ae)
---
src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java b/src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java
index a32cc2a..5391b3d 100644
--- a/src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java
+++ b/src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java
@@ -305,11 +305,7 @@ public class IndyInterface {
);
if (NULL_METHOD_HANDLE_WRAPPER == mhw) {
- final MethodHandleWrapper fbMhw = fallbackSupplier.get();
- if (fbMhw.isCanSetTarget()) {
- doWithCallSite(callSite, arguments, (cs, receiver) -> cs.put(receiver.getClass().getName(), fbMhw));
- }
- mhw = fbMhw;
+ mhw = fallbackSupplier.get();
}
if (mhw.isCanSetTarget() && (callSite.getTarget() != mhw.getTargetMethodHandle()) && (mhw.getLatestHitCount() > INDY_OPTIMIZE_THRESHOLD)) {