[llvm-commits] [llvm] r47980 - /llvm/trunk/lib/Transforms/IPO/StructRetPromotion.cpp

[Devang Patel]

Author: dpatel
Date: Wed Mar  5 17:39:23 2008
New Revision: 47980

URL: http://llvm.org/viewvc/llvm-project?rev=47980&view=rev
Log:
Skip, for now, callsites where use of sret argument is not dominated by callsite.

Modified:
    llvm/trunk/lib/Transforms/IPO/StructRetPromotion.cpp

Modified: llvm/trunk/lib/Transforms/IPO/StructRetPromotion.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/StructRetPromotion.cpp?rev=47980&r1=47979&r2=47980&view=diff

==============================================================================
--- llvm/trunk/lib/Transforms/IPO/StructRetPromotion.cpp (original)
+++ llvm/trunk/lib/Transforms/IPO/StructRetPromotion.cpp Wed Mar  5 17:39:23 2008
@@ -171,6 +171,10 @@
       // If FirstArg user is a GEP whose all users are not LoadInst then
       // this function F is not suitable for sret promotion.
       else if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(ArgI)) {
+        // TODO : Use dom info and insert PHINodes to collect get results
+        // from multiple call sites for this GEP.
+        if (GEP->getParent() != Call->getParent())
+          return false;
         for (Value::use_iterator GEPI = GEP->use_begin(), GEPE = GEP->use_end();
              GEPI != GEPE; ++GEPI) 
           if (!isa<LoadInst>(GEPI))