ptx: fix lint and compiler warnings
authorChe-Liang Chiou <clchiou@gmail.com>
Wed, 2 Mar 2011 07:58:46 +0000 (07:58 +0000)
committerChe-Liang Chiou <clchiou@gmail.com>
Wed, 2 Mar 2011 07:58:46 +0000 (07:58 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@126838 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/PTX/PTXAsmPrinter.cpp
lib/Target/PTX/PTXTargetMachine.cpp
lib/Target/PTX/PTXTargetMachine.h

index 2249d86830502e246a37d21fd0fbcd99a631499f..13134032cb7a32e2206d4e163a6cf07d995448cc 100644 (file)
@@ -361,7 +361,8 @@ void PTXAsmPrinter::EmitFunctionDeclaration() {
       unsigned cnt = 0;
       //for (int i = 0, e = MFI->getNumArg(); i != e; ++i) {
       for(PTXMachineFunctionInfo::reg_reverse_iterator
-          i = MFI->argRegReverseBegin(), e = MFI->argRegReverseEnd(), b = i; i != e; ++i) {
+          i = MFI->argRegReverseBegin(), e = MFI->argRegReverseEnd(), b = i;
+          i != e; ++i) {
         reg = *i;
         assert(reg != PTX::NoRegister && "Not a valid register!");
         if (i != b)
@@ -374,7 +375,8 @@ void PTXAsmPrinter::EmitFunctionDeclaration() {
       }
     } else {
       for (PTXMachineFunctionInfo::reg_reverse_iterator
-           i = MFI->argRegReverseBegin(), e = MFI->argRegReverseEnd(), b = i; i != e; ++i) {
+           i = MFI->argRegReverseBegin(), e = MFI->argRegReverseEnd(), b = i;
+           i != e; ++i) {
         reg = *i;
         assert(reg != PTX::NoRegister && "Not a valid register!");
         if (i != b)
index 371510fe3250fb86a551e49774929ba1760d6b9c..4701a941d181806845a37228b080a8d413b331cc 100644 (file)
@@ -36,23 +36,25 @@ extern "C" void LLVMInitializePTXTarget() {
 }
 
 namespace {
-  const char* DataLayout32 = "e-p:32:32-i64:32:32-f64:32:32-v128:32:128-v64:32:64-n32:64";
-  const char* DataLayout64 = "e-p:64:64-i64:32:32-f64:32:32-v128:32:128-v64:32:64-n32:64";
+  const char* DataLayout32 =
+    "e-p:32:32-i64:32:32-f64:32:32-v128:32:128-v64:32:64-n32:64";
+  const char* DataLayout64 =
+    "e-p:64:64-i64:32:32-f64:32:32-v128:32:128-v64:32:64-n32:64";
 }
 
 // DataLayout and FrameLowering are filled with dummy data
 PTXTargetMachine::PTXTargetMachine(const Target &T,
                                    const std::string &TT,
                                    const std::string &FS)
-  : Subtarget(TT, FS),
+  : LLVMTargetMachine(T, TT),
     // FIXME: This feels like a dirty hack, but Subtarget does not appear to be
     //        initialized at this point, and we need to finish initialization of
     //        DataLayout.
     DataLayout((FS.find("64bit") != FS.npos) ? DataLayout64 : DataLayout32),
-    LLVMTargetMachine(T, TT),
+    Subtarget(TT, FS),
     FrameLowering(Subtarget),
-    TLInfo(*this),
-    InstrInfo(*this) {
+    InstrInfo(*this),
+    TLInfo(*this) {
 }
 
 bool PTXTargetMachine::addInstSelector(PassManagerBase &PM,
index 2add7134baef8f24694b5a03109d5209e47fb7b6..a5dba537d1dfc14843c676846bbd0add45d5cefe 100644 (file)
@@ -26,10 +26,10 @@ namespace llvm {
 class PTXTargetMachine : public LLVMTargetMachine {
   private:
     const TargetData  DataLayout;
+    PTXSubtarget      Subtarget; // has to be initialized before FrameLowering
     PTXFrameLowering  FrameLowering;
     PTXInstrInfo      InstrInfo;
     PTXTargetLowering TLInfo;
-    PTXSubtarget      Subtarget;
 
   public:
     PTXTargetMachine(const Target &T, const std::string &TT,