Issue Details (XML | Word | Printable)

Key: RVM-195
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Ian Rogers
Reporter: Ian Rogers
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
RVM

Enable inlining of @Pure methods

Created: 25/Aug/07 09:44 AM   Updated: 23/Jul/08 11:46 AM
Component/s: Compiler: Optimizing
Affects Version/s: None
Fix Version/s: 2.9.3

Time Tracking:
Not Specified

Issue Links:
Related
 


 Description  « Hide
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.

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Ian Rogers added a comment - 14/Mar/08 04:22 AM
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.