Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: jruby/jruby
base: 5e46bfc340ca
Choose a base ref
...
head repository: jruby/jruby
compare: 652664323c42
Choose a head ref
  • 3 commits
  • 114 files changed
  • 1 contributor

Commits on Jan 12, 2015

  1. Convert all instrs to store the operands and result to instr class.

    This work eliminates a lot of tmeporary array construction of getOperands[] by storing
    all operands in instr by default.  It adds a small amount of memory for this aspect (basically
    an extra primitive array ref for all non d*node instrs/call methods).  Unfortunately, I also
    pushed result down to instr and this ended up using more memory that I had hoped for.  A
    followup commit will address this.  Once result is fixed this will use ~500-750k more memory
    than before this commit for rails console.
    
    The secondary benefit to this work is it eliminates quite a few repeated methods in instrs.
    Methods like simplifyOperand end up being a single method on instr now.
    enebo committed Jan 12, 2015
    Copy the full SHA
    c34f31c View commit details
    Browse the repository at this point in the history
  2. Copy the full SHA
    c9faa41 View commit details
    Browse the repository at this point in the history
  3. Copy the full SHA
    6526643 View commit details
    Browse the repository at this point in the history