[llvm-commits] [llvm] r102316 - /llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp

[Dale Johannesen]

Author: johannes
Date: Sun Apr 25 16:03:54 2010
New Revision: 102316

URL: http://llvm.org/viewvc/llvm-project?rev=102316&view=rev
Log:
Add comment re byval args.  Doesn't actually work this way yet.
xs

Modified:
    llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp?rev=102316&r1=102315&r2=102316&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp Sun Apr 25 16:03:54 2010
@@ -340,6 +340,8 @@
       return true;
     const AllocaInst *AI = dyn_cast<AllocaInst>(Address);
     // Don't handle byval struct arguments or VLAs, for example.
+    // Note that if we have a byval struct argument, fast ISel is turned off;
+    // those are handled in SelectionDAGBuilder.
     if (!AI) break;
     DenseMap<const AllocaInst*, int>::iterator SI =
       StaticAllocaMap.find(AI);
@@ -347,7 +349,7 @@
     int FI = SI->second;
     if (!DI->getDebugLoc().isUnknown())
       MF.getMMI().setVariableDbgInfo(DI->getVariable(), FI, DI->getDebugLoc());
-    
+
     // Building the map above is target independent.  Generating DBG_VALUE
     // inline is target dependent; do this now.
     (void)TargetSelectInstruction(cast<Instruction>(I));