Don't barf on empty basic blocks. Do not rely on assert
[oota-llvm.git] / lib / VMCore / AutoUpgrade.cpp
index 91b0d31dff435fd595f0a7fc5d5cca5b1e150286..b56fe70235e031a1b167f0afb630167a1b83814d 100644 (file)
@@ -2,7 +2,7 @@
 //
 //                     The LLVM Compiler Infrastructure
 //
-// This file was developed by Reid Spencer and is distributed under the 
+// This file was developed by Chandler Carruth and is distributed under the 
 // University of Illinois Open Source License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 //
 //===----------------------------------------------------------------------===//
 
-#include "llvm/Assembly/AutoUpgrade.h"
-#include "llvm/DerivedTypes.h"
+#include "llvm/AutoUpgrade.h"
 #include "llvm/Function.h"
 #include "llvm/Module.h"
 #include "llvm/Instructions.h"
+#include "llvm/ParameterAttributes.h"
 #include "llvm/Intrinsics.h"
-#include "llvm/SymbolTable.h"
-#include <iostream>
-
 using namespace llvm;
 
-// Utility function for getting the correct suffix given a type
-static inline const char* get_suffix(const Type* Ty) {
-  switch (Ty->getTypeID()) {
-    case Type::UIntTyID:    return ".i32";
-    case Type::UShortTyID:  return ".i16";
-    case Type::UByteTyID:   return ".i8";
-    case Type::ULongTyID:   return ".i64";
-    case Type::FloatTyID:   return ".f32";
-    case Type::DoubleTyID:  return ".f64";
-    default:                break;                        
-  }
-  return 0;
-}
 
-static inline const Type* getTypeFromFunctionName(Function* F) {
-  // If there's no function, we can't get the argument type.
-  if (!F)
-    return 0;
+Function* llvm::UpgradeIntrinsicFunction(Function *F) {
+  assert(F && "Illegal to upgrade a non-existent Function.");
 
   // Get the Function's name.
   const std::string& Name = F->getName();
 
+  // Convenience
+  const FunctionType *FTy = F->getFunctionType();
+
   // Quickly eliminate it, if it's not a candidate.
-  if (Name.length() <= 8 || Name[0] != 'l' || Name[1] != 'l' || Name[2] !=
-    'v' || Name[3] != 'm' || Name[4] != '.')
+  if (Name.length() <= 8 || Name[0] != 'l' || Name[1] != 'l' || 
+      Name[2] != 'v' || Name[3] != 'm' || Name[4] != '.')
     return 0;
 
+  Module *M = F->getParent();
   switch (Name[5]) {
-    case 'b':
-      if (Name == "llvm.bswap")
-        return F->getReturnType();
-      break;
-    case 'c':
-      if (Name == "llvm.ctpop" || Name == "llvm.ctlz" || Name == "llvm.cttz")
-        return F->getReturnType();
-      break;
-    case 'i':
-      if (Name == "llvm.isunordered") {
-        Function::const_arg_iterator ArgIt = F->arg_begin();
-        if (ArgIt != F->arg_end()) 
-          return ArgIt->getType();
+  default: break;
+  case 'b':
+    //  This upgrades the name of the llvm.bswap intrinsic function to only use 
+    //  a single type name for overloading. We only care about the old format
+    //  'llvm.bswap.i*.i*', so check for 'bswap.' and then for there being 
+    //  a '.' after 'bswap.'
+    if (Name.compare(5,6,"bswap.",6) == 0) {
+      std::string::size_type delim = Name.find('.',11);
+      
+      if (delim != std::string::npos) {
+        //  Construct the new name as 'llvm.bswap' + '.i*'
+        F->setName(Name.substr(0,10)+Name.substr(delim));
+        return F;
+      }
+    }
+    break;
+
+  case 'c':
+    //  We only want to fix the 'llvm.ct*' intrinsics which do not have the 
+    //  correct return type, so we check for the name, and then check if the 
+    //  return type does not match the parameter type.
+    if ( (Name.compare(5,5,"ctpop",5) == 0 ||
+          Name.compare(5,4,"ctlz",4) == 0 ||
+          Name.compare(5,4,"cttz",4) == 0) &&
+        FTy->getReturnType() != FTy->getParamType(0)) {
+      //  We first need to change the name of the old (bad) intrinsic, because 
+      //  its type is incorrect, but we cannot overload that name. We 
+      //  arbitrarily unique it here allowing us to construct a correctly named 
+      //  and typed function below.
+      F->setName("");
+
+      //  Now construct the new intrinsic with the correct name and type. We 
+      //  leave the old function around in order to query its type, whatever it 
+      //  may be, and correctly convert up to the new type.
+      return cast<Function>(M->getOrInsertFunction(Name, 
+                                                   FTy->getParamType(0),
+                                                   FTy->getParamType(0),
+                                                   (Type *)0));
+    }
+    break;
+
+  case 'p':
+    //  This upgrades the llvm.part.select overloaded intrinsic names to only 
+    //  use one type specifier in the name. We only care about the old format
+    //  'llvm.part.select.i*.i*', and solve as above with bswap.
+    if (Name.compare(5,12,"part.select.",12) == 0) {
+      std::string::size_type delim = Name.find('.',17);
+      
+      if (delim != std::string::npos) {
+        //  Construct a new name as 'llvm.part.select' + '.i*'
+        F->setName(Name.substr(0,16)+Name.substr(delim));
+        return F;
       }
       break;
-    case 's':
-      if (Name == "llvm.sqrt")
-        return F->getReturnType();
-      break;
-    default:
-      break;
-  }
-  return 0;
-}
-
-// This assumes the Function is one of the intrinsics we upgraded.
-static inline const Type* getTypeFromFunction(Function *F) {
-  const Type* Ty = F->getReturnType();
-  if (Ty->isFloatingPoint())
-    return Ty;
-  if (Ty->isSigned())
-    return Ty->getUnsignedVersion();
-  if (Ty->isInteger())
-    return Ty;
-  if (Ty == Type::BoolTy) {
-    Function::const_arg_iterator ArgIt = F->arg_begin();
-    if (ArgIt != F->arg_end()) 
-      return ArgIt->getType();
-  }
-  return 0;
-}
-
-bool llvm::IsUpgradeableIntrinsicName(const std::string& Name) {
-  // Quickly eliminate it, if it's not a candidate.
-  if (Name.length() <= 8 || Name[0] != 'l' || Name[1] != 'l' || Name[2] !=
-    'v' || Name[3] != 'm' || Name[4] != '.')
-    return false;
+    }
 
-  switch (Name[5]) {
-    case 'b':
-      if (Name == "llvm.bswap")
-        return true;
-      break;
-    case 'c':
-      if (Name == "llvm.ctpop" || Name == "llvm.ctlz" || Name == "llvm.cttz")
-        return true;
-      break;
-    case 'i':
-      if (Name == "llvm.isunordered")
-        return true;
-      break;
-    case 's':
-      if (Name == "llvm.sqrt")
-        return true;
-      break;
-    default:
+    //  This upgrades the llvm.part.set intrinsics similarly as above, however 
+    //  we care about 'llvm.part.set.i*.i*.i*', but only the first two types 
+    //  must match. There is an additional type specifier after these two 
+    //  matching types that we must retain when upgrading.  Thus, we require 
+    //  finding 2 periods, not just one, after the intrinsic name.
+    if (Name.compare(5,9,"part.set.",9) == 0) {
+      std::string::size_type delim = Name.find('.',14);
+
+      if (delim != std::string::npos &&
+          Name.find('.',delim+1) != std::string::npos) {
+        //  Construct a new name as 'llvm.part.select' + '.i*.i*'
+        F->setName(Name.substr(0,13)+Name.substr(delim));
+        return F;
+      }
       break;
-  }
-  return false;
-}
-
-// UpgradeIntrinsicFunction - Convert overloaded intrinsic function names to
-// their non-overloaded variants by appending the appropriate suffix based on
-// the argument types.
-Function* llvm::UpgradeIntrinsicFunction(Function* F) {
-  // See if its one of the name's we're interested in.
-  if (const Type* Ty = getTypeFromFunctionName(F)) {
-    const char* suffix = 
-      get_suffix((Ty->isSigned() ? Ty->getUnsignedVersion() : Ty));
-    assert(suffix && "Intrinsic parameter type not recognized");
-    const std::string& Name = F->getName();
-    std::string new_name = Name + suffix;
-    std::cerr << "WARNING: change " << Name << " to " << new_name << "\n";
-    SymbolTable& SymTab = F->getParent()->getSymbolTable();
-    if (Value* V = SymTab.lookup(F->getType(),new_name))
-      if (Function* OtherF = dyn_cast<Function>(V))
-        return OtherF;
-    
-    // There wasn't an existing function for the intrinsic, so now make sure the
-    // signedness of the arguments is correct.
-    if (Ty->isSigned()) {
-      const Type* newTy = Ty->getUnsignedVersion();
-      std::vector<const Type*> Params;
-      Params.push_back(newTy);
-      FunctionType* FT = FunctionType::get(newTy, Params,false);
-      return new Function(FT, GlobalValue::ExternalLinkage, new_name, 
-                          F->getParent());
     }
 
-    // The argument was the correct type (unsigned or floating), so just
-    // rename the function to its correct name and return it.
-    F->setName(new_name);
-    return F;
+    break;
   }
+
+  //  This may not belong here. This function is effectively being overloaded 
+  //  to both detect an intrinsic which needs upgrading, and to provide the 
+  //  upgraded form of the intrinsic. We should perhaps have two separate 
+  //  functions for this.
   return 0;
 }
 
+// UpgradeIntrinsicCall - Upgrade a call to an old intrinsic to be a call the 
+// upgraded intrinsic. All argument and return casting must be provided in 
+// order to seamlessly integrate with existing context.
+void llvm::UpgradeIntrinsicCall(CallInst *CI, Function *NewFn) {
+  assert(NewFn && "Cannot upgrade an intrinsic call without a new function.");
 
-Instruction* llvm::MakeUpgradedCall(
-    Function* F, const std::vector<Value*>& Params, BasicBlock* BB,
-    bool isTailCall, unsigned CallingConv) {
-  assert(F && "Need a Function to make a CallInst");
-  assert(BB && "Need a BasicBlock to make a CallInst");
-
-  // Convert the params
-  bool signedArg = false;
-  std::vector<Value*> Oprnds;
-  for (std::vector<Value*>::const_iterator PI = Params.begin(), 
-       PE = Params.end(); PI != PE; ++PI) {
-    const Type* opTy = (*PI)->getType();
-    if (opTy->isSigned()) {
-      signedArg = true;
-      CastInst* cast = 
-        new CastInst(*PI,opTy->getUnsignedVersion(), "autoupgrade_cast");
-      BB->getInstList().push_back(cast);
-      Oprnds.push_back(cast);
-    }
-    else
-      Oprnds.push_back(*PI);
-  }
-
-  Instruction* result = new CallInst(F,Oprnds,"autoupgrade_call");
-  if (isTailCall) cast<CallInst>(result)->setTailCall();
-  if (CallingConv) cast<CallInst>(result)->setCallingConv(CallingConv);
-  if (signedArg) {
-    const Type* newTy = F->getReturnType()->getUnsignedVersion();
-    CastInst* final = new CastInst(result, newTy, "autoupgrade_uncast");
-    BB->getInstList().push_back(result);
-    result = final;
-  }
-  return result;
-}
-
-Instruction* llvm::UpgradeIntrinsicCall(CallInst *CI, Function* newF) {
   Function *F = CI->getCalledFunction();
-  if (const Type* Ty = 
-      (newF ? getTypeFromFunction(newF) : getTypeFromFunctionName(F))) {
-    std::vector<Value*> Oprnds;
-    User::op_iterator OI = CI->op_begin(); 
-    ++OI;
-    for (User::op_iterator OE = CI->op_end() ; OI != OE; ++OI) {
-      const Type* opTy = OI->get()->getType();
-      if (opTy->isSigned())
-        Oprnds.push_back(
-          new CastInst(OI->get(),opTy->getUnsignedVersion(), 
-            "autoupgrade_cast",CI));
-      else
-        Oprnds.push_back(*OI);
+  assert(F && "CallInst has no function associated with it.");
+
+  const FunctionType *FTy = F->getFunctionType();
+  const FunctionType *NewFnTy = NewFn->getFunctionType();
+  
+  switch(NewFn->getIntrinsicID()) {
+  default:  assert(0 && "Unknown function for CallInst upgrade.");
+  case Intrinsic::ctlz:
+  case Intrinsic::ctpop:
+  case Intrinsic::cttz:
+    //  Build a small vector of the 1..(N-1) operands, which are the 
+    //  parameters.
+    SmallVector<Value*, 8>   Operands(CI->op_begin()+1, CI->op_end());
+
+    //  Construct a new CallInst
+    CallInst *NewCI = new CallInst(NewFn, Operands.begin(), Operands.end(), 
+                                   "upgraded."+CI->getName(), CI);
+    NewCI->setTailCall(CI->isTailCall());
+    NewCI->setCallingConv(CI->getCallingConv());
+
+    //  Handle any uses of the old CallInst.
+    if (!CI->use_empty()) {
+      //  Check for sign extend parameter attributes on the return values.
+      bool SrcSExt = NewFnTy->getParamAttrs() &&
+                     NewFnTy->getParamAttrs()->paramHasAttr(0,ParamAttr::SExt);
+      bool DestSExt = FTy->getParamAttrs() &&
+                      FTy->getParamAttrs()->paramHasAttr(0,ParamAttr::SExt);
+      
+      //  Construct an appropriate cast from the new return type to the old.
+      CastInst *RetCast = CastInst::create(
+                            CastInst::getCastOpcode(NewCI, SrcSExt,
+                                                    F->getReturnType(),
+                                                    DestSExt),
+                            NewCI, F->getReturnType(),
+                            NewCI->getName(), CI);
+      NewCI->moveBefore(RetCast);
+
+      //  Replace all uses of the old call with the new cast which has the 
+      //  correct type.
+      CI->replaceAllUsesWith(RetCast);
     }
-    CallInst* newCI = new CallInst((newF?newF:F),Oprnds,"autoupgrade_call",CI);
-    newCI->setTailCall(CI->isTailCall());
-    newCI->setCallingConv(CI->getCallingConv());
-    if (const Type* oldType = CI->getCalledFunction()->getReturnType())
-      if (oldType->isSigned()) {
-        CastInst* final = 
-          new CastInst(newCI, oldType, "autoupgrade_uncast",newCI);
-        newCI->moveBefore(final);
-        return final;
-      }
-    return newCI;
+
+    //  Clean up the old call now that it has been completely upgraded.
+    CI->eraseFromParent();
+    break;
   }
-  return 0;
 }
 
-bool llvm::UpgradeCallsToIntrinsic(Function* F) {
-  if (Function* newF = UpgradeIntrinsicFunction(F)) {
-    for (Value::use_iterator UI = F->use_begin(), UE = F->use_end();
-         UI != UE; ) {
-      if (CallInst* CI = dyn_cast<CallInst>(*UI++)) {
-        std::vector<Value*> Oprnds;
-        User::op_iterator OI = CI->op_begin();
-        ++OI;
-        for (User::op_iterator OE = CI->op_end(); OI != OE; ++OI) {
-          const Type* opTy = OI->get()->getType();
-          if (opTy->isSigned()) {
-            Oprnds.push_back(
-              new CastInst(OI->get(),opTy->getUnsignedVersion(), 
-                  "autoupgrade_cast",CI));
-          }
-          else
-            Oprnds.push_back(*OI);
-        }
-        CallInst* newCI = new CallInst(newF,Oprnds,"autoupgrade_call",CI);
-        newCI->setTailCall(CI->isTailCall());
-        newCI->setCallingConv(CI->getCallingConv());
-        if (const Type* Ty = CI->getCalledFunction()->getReturnType())
-          if (Ty->isSigned()) {
-            CastInst* final = 
-              new CastInst(newCI, Ty, "autoupgrade_uncast",newCI);
-            newCI->moveBefore(final);
-            CI->replaceAllUsesWith(final);
-          } else {
-            CI->replaceAllUsesWith(newCI);
-          }
-        CI->eraseFromParent();
+// This tests each Function to determine if it needs upgrading. When we find 
+// one we are interested in, we then upgrade all calls to reflect the new 
+// function.
+void llvm::UpgradeCallsToIntrinsic(Function* F) {
+  assert(F && "Illegal attempt to upgrade a non-existent intrinsic.");
+
+  // Upgrade the function and check if it is a totaly new function.
+  if (Function* NewFn = UpgradeIntrinsicFunction(F)) {
+    if (NewFn != F) {
+      // Replace all uses to the old function with the new one if necessary.
+      for (Value::use_iterator UI = F->use_begin(), UE = F->use_end();
+           UI != UE; ) {
+        if (CallInst* CI = dyn_cast<CallInst>(*UI++))
+          UpgradeIntrinsicCall(CI, NewFn);
       }
-    }
-    if (newF != F)
+      // Remove old function, no longer used, from the module.
       F->eraseFromParent();
-    return true;
+    }
   }
-  return false;
 }
+