RVM

Enable inlining of @Pure methods

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.9.3
  • Component/s: Compiler: Optimizing
  • Labels:
    None
  • Number of attachments :
    0

Description

It should be reasonably straightforward to enable inlining of @Pure methods during expansion of runtime services. I believe having an annotation parameter controlling this makes sense, the default value of this annotation should be to inline @Pure methods when simplification has failed.

Issue Links

This issue relates to:
RVM-394 Tune inlining of arraycopy Major Closed

Activity

Hide
Permalink
Ian Rogers added a comment -

We currently inline Pure methods after first trying to simplify them in BC2IR. If we inline early we may miss opportunities to simplify. Perhaps we want to postpone inlining Pure methods, or be able to recover that a region of code was a pure method. It's not a simple picture and I think as of r14027 we're in a sensible shape without a major rethink.

Show
Ian Rogers added a comment - We currently inline Pure methods after first trying to simplify them in BC2IR. If we inline early we may miss opportunities to simplify. Perhaps we want to postpone inlining Pure methods, or be able to recover that a region of code was a pure method. It's not a simple picture and I think as of r14027 we're in a sensible shape without a major rethink.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.