Compute the loop nest depth of each CfgNode and weight Variables by it.

src/IceInst.h

 //===- subzero/src/IceInst.h - High-level instructions ----------*- C++ -*-===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 ///
 /// \file
 /// This file declares the Inst class and its target-independent
 /// subclasses, which represent the high-level Vanilla ICE instructions
 /// and map roughly 1:1 to LLVM instructions.
 ///
 //===----------------------------------------------------------------------===//
 
 #ifndef SUBZERO_SRC_ICEINST_H
 #define SUBZERO_SRC_ICEINST_H
 
 #include "IceCfg.h"
 #include "IceDefs.h"
 #include "IceInst.def"
 #include "IceIntrinsics.h"
+#include "IceOperand.h"

[Jim Stichnoth, 2015/09/03 21:35:09]

Why is this needed?

[ascull, 2015/09/03 22:47:12]

Line 696 has a dyn_cast to Variable which requires the complete type. I came
across this when including IceDefs from LoopAnalyzer.

[Jim Stichnoth, 2015/09/03 23:23:30]

Ah, thanks.  In that case, could you move the definition of getRmwBeacon into
IceInst.cpp, so that this include isn't needed?

[ascull, 2015/09/04 00:23:51]

Done.

 #include "IceTypes.h"
 
 // TODO: The Cfg structure, and instructions in particular, need to be
 // validated for things like valid operand types, valid branch
 // targets, proper ordering of Phi and non-Phi instructions, etc.
 // Most of the validity checking will be done in the bitcode reader.
 // We need a list of everything that should be validated, and tests
 // for each.
 
 namespace Ice {
@@ skipping 948 matching lines @@
   static void noteHead(Ice::Inst *, Ice::Inst *) {}
   void deleteNode(Ice::Inst *) {}
 
 private:
   mutable ilist_half_node<Ice::Inst> Sentinel;
 };
 
 } // end of namespace llvm
 
 #endif // SUBZERO_SRC_ICEINST_H