Changes the TargetX8632 to inherit from TargetX86Base<TargetX8632>.

src/IceTargetLoweringX8632.h

 //===- subzero/src/IceTargetLoweringX8632.h - x86-32 lowering ---*- 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 TargetLoweringX8632 class, which
 /// implements the TargetLowering interface for the x86-32
 /// architecture.
 ///
 //===----------------------------------------------------------------------===//
 
 #ifndef SUBZERO_SRC_ICETARGETLOWERINGX8632_H
 #define SUBZERO_SRC_ICETARGETLOWERINGX8632_H
 
 #include "IceAssemblerX8632.h"
 #include "IceDefs.h"
 #include "IceInstX8632.h"
 #include "IceRegistersX8632.h"
 #include "IceTargetLowering.h"
 #include "IceTargetLoweringX8632Traits.h"
+#include "IceTargetLoweringX86Base.h"
 
 namespace Ice {
 
-class TargetX8632 : public TargetLowering {
+class TargetX8632 final
+    : public ::Ice::X86Internal::TargetX86Base<TargetX8632> {
   TargetX8632() = delete;
   TargetX8632(const TargetX8632 &) = delete;
   TargetX8632 &operator=(const TargetX8632 &) = delete;
 
 public:
   using X86InstructionSet = X8632::Traits::InstructionSet;
 
-  static TargetX8632 *create(Cfg *Func);
-  virtual X8632::Traits::Address
-  stackVarToAsmOperand(const Variable *Var) const = 0;
-  virtual X86InstructionSet getInstructionSet() const = 0;
+  static TargetX8632 *create(Cfg *Func) { return new TargetX8632(Func); }
 
 protected:
-  explicit TargetX8632(Cfg *Func) : TargetLowering(Func) {}
+  Operand *createNaClReadTPSrcOperand() {
+    Constant *Zero = Ctx->getConstantZero(IceType_i32);
+    return Traits::X86OperandMem::create(Func, IceType_i32, nullptr, Zero,

[jvoung (off chromium), 2015/07/08 18:50:11]

So the X8664 version would be something like:

Create a TmpDest variable, lowerCall("naclreadtp", TmpDest), return TmpDest?

[John, 2015/07/08 21:44:41]

I don't know, yet -- mostly because I don't know how this will be lowered in
X8664, but you got the idea.

+                                         nullptr, 0,
+                                         Traits::X86OperandMem::SegReg_GS);
+  }
+
+private:
+  friend class ::Ice::X86Internal::TargetX86Base<TargetX8632>;

[jvoung (off chromium), 2015/07/08 18:50:11]

So this is to allow the parent dispatch to methods defined here in the child?

[John, 2015/07/08 21:44:41]

Yes, exactly. Or we could make those methods public -- I'd rather use friendship
here.

+
+  explicit TargetX8632(Cfg *Func) : TargetX86Base(Func) {}
 };
 
 class TargetDataX8632 final : public TargetDataLowering {
   TargetDataX8632() = delete;
   TargetDataX8632(const TargetDataX8632 &) = delete;
   TargetDataX8632 &operator=(const TargetDataX8632 &) = delete;
 
 public:
   static std::unique_ptr<TargetDataLowering> create(GlobalContext *Ctx) {
     return std::unique_ptr<TargetDataLowering>(new TargetDataX8632(Ctx));
@@ skipping 24 matching lines @@
 protected:
   explicit TargetHeaderX8632(GlobalContext *Ctx);
 
 private:
   ~TargetHeaderX8632() = default;
 };
 
 } // end of namespace Ice
 
 #endif // SUBZERO_SRC_ICETARGETLOWERINGX8632_H