Skip to content

Conversation

@woess
Copy link
Member

@woess woess commented Jan 25, 2016

just some javadoc cleanup and object model access modifier fixes in the implementation package.

@smarr
Copy link
Contributor

smarr commented Jan 25, 2016

Thanks for the visibility changes.
And, looks good to me.

@woess woess merged commit 2341199 into oracle:master Jan 25, 2016
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know.

dougxc pushed a commit that referenced this pull request Apr 21, 2016
…OM/truffle:master to master

* commit 'feec4a84e5157b24c01329c341c69b389ecdd065':
  Factor language check into guard helper method in CachedObjectAccessNode
  Rename LanguageCheck annotation to CanResolve
  Use deep copy of can-resolve node of interop DSL
  Add tests for @Languagecheck annotation
  Extend Interop DSL and provide an @Languagecheck annotation, which allows expressing the language check as a node
sanzinger pushed a commit to XiaohongGong/graal that referenced this pull request Jan 7, 2020
We want to find a better way to implement array offset calculation.
The old pattern for an int array member loading is:
       and      x4, x3, #0xffffffff
       lsl      x4, x4, #2
       add      x4, x4, #0x10
       ldr      x0, [x2, x4]

And the optimized pattern is:
       add      x1, x2, w3, uxtw #2
       ldr      x0, [x1, oracle#16]

Here is a jmh unit test for int array member loading:
    private final int func(int[] arr) {
        int ret = 0;
        for (int i=0; i < field0; i++) {
            ret += arr[i];
        }
        return ret;
    }
And the jmh performance results:
     without this patch   with this patch   units
         1173501.595        948290.420      us/op

Here are the additional issues fixed related to this patch:
1. Fix an issue in MatchProcessor, so that we can process platform specific node
as a matchable node.

2. Close this optimization for substratevm because the derived reference is not
supported yet in SVM.

mx bootstrap passes.
mx gate -o -t 'CTW:hosted' passes.

Change-Id: Ia34f2bf044ca6863be075cf000b29524ca44c42d
jerboaa pushed a commit to jerboaa/graal that referenced this pull request Aug 27, 2024
Partial backport of [GR-48529] Use the API flag in the init hint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants