Subzero: Remove the IceV_RegManager enum value.

This hasn't been used in a very long time, and there's no intention of using it again. Originally there was the idea of a "fast" block-local register allocator for an O1-like configuration, which would allocate registers for infinite-weight temporaries during target lowering, using a "local register manager". This verbose option was for tracing execution of this register manager. However, by now it seems unlikely that this would do a better/faster job than the current Om1 register allocation approach, which reuses the linear-scan code quite effectively and does very well at separation of concerns. So adios IceV_RegManager! BUG= none R=jvoung@chromium.org Review URL: https://codereview.chromium.org/831663008

Subzero: Remove the IceV_RegManager enum value.
769be681 · Jim Stichnoth · ec270731 · 769be681 · 769be681
Commit 769be681 authored Jan 15, 2015 by Jim Stichnoth
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 7 deletions

IceDefs.h src/IceDefs.h +5 -6

llvm2ice.cpp src/llvm2ice.cpp +0 -1

No files found.
--- a/src/IceDefs.h
+++ b/src/IceDefs.h
@@ -148,12 +148,11 @@ enum VerboseItem {
  IceV_Preds = 1 << 3,
  IceV_Succs = 1 << 4,
  IceV_Liveness = 1 << 5,
-  IceV_RegManager = 1 << 6,
-  IceV_RegOrigins = 1 << 7,
-  IceV_LinearScan = 1 << 8,
-  IceV_Frame = 1 << 9,
-  IceV_AddrOpt = 1 << 10,
-  IceV_Random = 1 << 11,
+  IceV_RegOrigins = 1 << 6,
+  IceV_LinearScan = 1 << 7,
+  IceV_Frame = 1 << 8,
+  IceV_AddrOpt = 1 << 9,
+  IceV_Random = 1 << 10,
  IceV_All = ~IceV_None,
  IceV_Most = IceV_All & ~IceV_LinearScan
 };

--- a/src/llvm2ice.cpp
+++ b/src/llvm2ice.cpp
@@ -44,7 +44,6 @@ static cl::list<Ice::VerboseItem> VerboseList(
        clEnumValN(Ice::IceV_Preds, "pred", "Show predecessors"),
        clEnumValN(Ice::IceV_Succs, "succ", "Show successors"),
        clEnumValN(Ice::IceV_Liveness, "live", "Liveness information"),
-        clEnumValN(Ice::IceV_RegManager, "rmgr", "Register manager status"),
        clEnumValN(Ice::IceV_RegOrigins, "orig", "Physical register origins"),
        clEnumValN(Ice::IceV_LinearScan, "regalloc", "Linear scan details"),
        clEnumValN(Ice::IceV_Frame, "frame", "Stack frame layout details"),