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

[Chris Lattner]

Changes in directory llvm/tools/bugpoint:

ExtractFunction.cpp updated: 1.25 -> 1.26

---
Log message:

add a fixme


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

Index: llvm/tools/bugpoint/ExtractFunction.cpp
diff -u llvm/tools/bugpoint/ExtractFunction.cpp:1.25 llvm/tools/bugpoint/ExtractFunction.cpp:1.26
--- llvm/tools/bugpoint/ExtractFunction.cpp:1.25	Sun Mar 14 13:27:19 2004
+++ llvm/tools/bugpoint/ExtractFunction.cpp	Sun Mar 14 13:31:00 2004
@@ -144,6 +144,11 @@
 /// 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