RVM
  1. RVM
  2. RVM-273 Remove VM_ prefix from all VM classes
  3. RVM-518

Change build logic so that the VM prefix on a class name does not control inclusion in the bootimage

    Details

    • Type: Sub-task Sub-task
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0
    • Component/s: Infrastructure: Build
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Historically the VM* name meant bootimage inclusion. We've partially moved away from that, but there are still a few remnants. Get rid of them while maintaining approximately the same set of classes included in the bootimage.

      With the change I'm testing now, the only significant difference is that a bunch of opt-compiler specific classes from ArchitectureSpecific get sucked into bootimage that don't include the opt compiler. This isn't ideal, but I think we can live with it for now. As we reduce usage of ArchitectureSpecific in the opt compiler (which we really should be doing anyways), this will go away. For the record, the list of classes is:

      > Lorg/jikesrvm/ArchitectureSpecific;
      > Lorg/jikesrvm/ArchitectureSpecific$Assembler;
      > Lorg/jikesrvm/ArchitectureSpecific$BURS_Debug;
      > Lorg/jikesrvm/ArchitectureSpecific$BURS_STATE;
      > Lorg/jikesrvm/ArchitectureSpecific$BURS_TreeNode;
      > Lorg/jikesrvm/ArchitectureSpecific$CallingConvention;
      > Lorg/jikesrvm/ArchitectureSpecific$ComplexLIR2MIRExpansion;
      > Lorg/jikesrvm/ArchitectureSpecific$ConvertALUOperators;
      > Lorg/jikesrvm/ArchitectureSpecific$FinalMIRExpansion;
      > Lorg/jikesrvm/ArchitectureSpecific$GenerateMachineSpecificMagic;
      > Lorg/jikesrvm/ArchitectureSpecific$MIROptimizationPlanner;
      > Lorg/jikesrvm/ArchitectureSpecific$NormalizeConstants;
      > Lorg/jikesrvm/ArchitectureSpecific$PhysicalDefUse;
      > Lorg/jikesrvm/ArchitectureSpecific$PhysicalRegisterConstants;
      > Lorg/jikesrvm/ArchitectureSpecific$PhysicalRegisterSet;
      > Lorg/jikesrvm/ArchitectureSpecific$PhysicalRegisterTools;
      > Lorg/jikesrvm/ArchitectureSpecific$RegisterPool;
      > Lorg/jikesrvm/ArchitectureSpecific$RegisterPreferences;
      > Lorg/jikesrvm/ArchitectureSpecific$RegisterRestrictions;
      > Lorg/jikesrvm/ArchitectureSpecific$StackManager;

        Activity

        Hide
        David Grove added a comment -

        fix committed in r14343.

        Show
        David Grove added a comment - fix committed in r14343.
        David Grove made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Closed [ 6 ]
        Hide
        David Grove added a comment -

        reopening so I can modify fix target to 3.0

        Show
        David Grove added a comment - reopening so I can modify fix target to 3.0
        David Grove made changes -
        Resolution Fixed [ 1 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        David Grove made changes -
        Fix Version/s 3.0 [ 13530 ]
        Fix Version/s 2.9.4 [ 14162 ]
        David Grove made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            David Grove
            Reporter:
            David Grove
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: