[llvm-commits] CVS: llvm/tools/bugpoint/ExtractFunction.cpp

Patrick Jenkins pjenkins at apple.com
Fri Jul 28 11:03:15 PDT 2006



Changes in directory llvm/tools/bugpoint:

ExtractFunction.cpp updated: 1.49 -> 1.50
---
Log message:

Updating the comments above SplitFunctionsOutOfModule in ExtractFunction.cpp to reflect the changes made to that function. Specifically I am removing the FIXME comment because the issue has been addressed. 


---
Diffs of the changes:  (+0 -5)

 ExtractFunction.cpp |    5 -----
 1 files changed, 5 deletions(-)


Index: llvm/tools/bugpoint/ExtractFunction.cpp
diff -u llvm/tools/bugpoint/ExtractFunction.cpp:1.49 llvm/tools/bugpoint/ExtractFunction.cpp:1.50
--- llvm/tools/bugpoint/ExtractFunction.cpp:1.49	Thu Jul 27 20:19:28 2006
+++ llvm/tools/bugpoint/ExtractFunction.cpp	Fri Jul 28 13:03:01 2006
@@ -312,11 +312,6 @@
 /// SplitFunctionsOutOfModule - Given a module and a list of functions in the
 /// module, split the functions OUT of the specified module, and place them in
 /// the new module.
-///
-/// FIXME: this could be made DRAMATICALLY more efficient for large programs if
-/// we just MOVED functions from one module to the other, instead of cloning the
-/// whole module, then proceeding to delete an entire module's worth of stuff.
-///
 Module *llvm::SplitFunctionsOutOfModule(Module *M,
                                         const std::vector<Function*> &F) {
   // Make sure functions & globals are all external so that linkage






More information about the llvm-commits mailing list