[llvm-commits] CVS: llvm/include/llvm/Assembly/AutoUpgrade.h

[Reid Spencer]

Changes in directory llvm/include/llvm/Assembly:

AutoUpgrade.h updated: 1.1 -> 1.2
---
Log message:

Add a new interface function to AutoUpgrade for simultaneously upgrading
the Function and the CallInst: UpgradeCallsToIntrinsic(Function*). Also,
re-factor the AutoUpgrade implementation to eliminate some duplication of
code.


---
Diffs of the changes:  (+25 -1)

 AutoUpgrade.h |   26 +++++++++++++++++++++++++-
 1 files changed, 25 insertions(+), 1 deletion(-)


Index: llvm/include/llvm/Assembly/AutoUpgrade.h
diff -u llvm/include/llvm/Assembly/AutoUpgrade.h:1.1 llvm/include/llvm/Assembly/AutoUpgrade.h:1.2
--- llvm/include/llvm/Assembly/AutoUpgrade.h:1.1	Mon Jan 16 15:06:01 2006
+++ llvm/include/llvm/Assembly/AutoUpgrade.h	Wed Jan 18 19:18:29 2006
@@ -14,8 +14,17 @@
 #ifndef LLVM_ASSEMBLY_AUTOUPGRADE_H
 #define LLVM_ASSEMBLY_AUTOUPGRADE_H
 
+#include <string>
+
 namespace llvm {
   class Function;
+  class CallInst;
+
+  /// This function determines if the \p Name provides is a name for which the
+  /// auto-upgrade to a non-overloaded name applies.
+  /// @returns True if the function name is upgradeable, false otherwise.
+  /// @brief Determine if a name is an upgradeable intrinsic name.
+  bool IsUpgradeableIntrinsicName(const std::string& Name);
 
   /// This function inspects the Function \p F to see if it is an old overloaded
   /// intrinsic. If it is, the Function's name is changed to add a suffix that
@@ -25,8 +34,23 @@
   /// the auto-upgrade feature from the old overloaded names to the new
   /// non-overloaded names. 
   /// @param F The Function to potentially auto-upgrade.
+  /// @returns A corrected version of F, or 0 if no change necessary
   /// @brief Remove overloaded intrinsic function names.
-  bool UpgradeIntrinsicFunction(Function* F);
+  Function* UpgradeIntrinsicFunction(Function* F);
+
+  /// This function inspects the CallInst \p CI to see if it is a call to an
+  /// old overloaded intrinsic. If it is, the CallInst's name is changed to add
+  /// a suffix that indicates the kind of arguments or result that it accepts.
+  /// In LLVM 1.7, the overloading of intrinsic functions was replaced with
+  /// separate functions for each of the various argument sizes. This function
+  /// implements the auto-upgrade feature from old overloaded names to the new
+  /// non-overloaded names.
+  /// @param CI The CallInst to potentially auto-upgrade.
+  /// @returns True if the call was upgraded, false otherwise.
+  /// @brief Replace overloaded intrinsic function calls.
+  CallInst* UpgradeIntrinsicCall(CallInst* CI);
+
+  bool UpgradeCallsToIntrinsic(Function* F);
 
 } // End llvm namespace