Changes in directory llvm/include/llvm/Assembly:
AutoUpgrade.h updated: 1.2 -> 1.3 --- Log message: 1. Documentation upgrade. 2. Have UpgradeInstrinicCall return an Instruction* instead of a CallInst* and return the needed CastInst* if the result of the upgraded call needs to be casted back to a signed type. --- Diffs of the changes: (+17 -8) AutoUpgrade.h | 25 +++++++++++++++++-------- 1 files changed, 17 insertions(+), 8 deletions(-) Index: llvm/include/llvm/Assembly/AutoUpgrade.h diff -u llvm/include/llvm/Assembly/AutoUpgrade.h:1.2 llvm/include/llvm/Assembly/AutoUpgrade.h:1.3 --- llvm/include/llvm/Assembly/AutoUpgrade.h:1.2 Wed Jan 18 19:18:29 2006 +++ llvm/include/llvm/Assembly/AutoUpgrade.h Thu Jan 19 00:59:26 2006 @@ -19,6 +19,7 @@ namespace llvm { class Function; class CallInst; + class Instruction; /// This function determines if the \p Name provides is a name for which the /// auto-upgrade to a non-overloaded name applies. @@ -38,18 +39,26 @@ /// @brief Remove overloaded intrinsic function names. 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. + /// non-overloaded names. This function inspects the CallInst \p CI to see + /// if it is a call to an old overloaded intrinsic. If it is, a new CallInst + /// is created that uses the correct Function and possibly casts the + /// argument and result to an unsigned type. The caller can use the + /// returned Instruction to replace the existing one. Note that the + /// Instruction* returned could be a CallInst or a CastInst depending on + /// whether casting was necessary. /// @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); - + /// @returns An instrution to replace \p CI with. + /// @brief Get replacement instruction for overloaded intrinsic function call. + Instruction* UpgradeIntrinsicCall(CallInst* CI); + + /// Upgrade both the function and all the calls made to it, if that function + /// needs to be upgraded. This is like a combination of the above two + /// functions, UpgradeIntrinsicFunction and UpgradeIntrinsicCall. Note that + /// the calls are replaced so this should only be used in a post-processing + /// manner (i.e. after all assembly/bytecode has been read). bool UpgradeCallsToIntrinsic(Function* F); } // End llvm namespace _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits