Uploaded image for project: 'JikesRVM'
  1. JikesRVM
  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
    • Status: Closed
    • Priority: Medium
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0
    • Component/s: Infrastructure: Build
    • Labels:
      None

      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;

        Attachments

          Activity

          dgrove David Grove created issue -
          Hide
          dgrove David Grove added a comment -

          fix committed in r14343.

          Show
          dgrove David Grove added a comment - fix committed in r14343.
          Hide
          dgrove David Grove added a comment -

          reopening so I can modify fix target to 3.0

          Show
          dgrove David Grove added a comment - reopening so I can modify fix target to 3.0
          dgrove David Grove made changes -
          Field Original Value New Value
          Workflow jira [ 17286 ] X10 Workflow [ 18523 ]
          dgrove David Grove made changes -
          Priority Major [ 6 ] Medium [ 3 ]
          ebrangs Erik Brangs made changes -
          Parent RVM-273 [ 13999 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: