[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