search for: calculatetailcallargdest

...ameInfo > * MFI) { > > I wonder if these can be moved to SelectionDAGISel? So it's handled > during the sdisel phase of call lowering? I am not sure what you mean by 'these' here. The function above (IsPossiblyOverwritten...) is used in LowerCALL() forall arguments CalculateTailCallArgDest() to determine whether an argument needs to be evacuated to a virtual register to prevent being overwritten. I am not sure what you are suggesting ? Storing this information (ispossiblyoverwritten) in the argument nodes? as ARG_FLAGSSDNode as a new ArgFlagsTy? You are using 'these'. I bel...

...PR[GPR_idx++], Arg)); } else { - MemOpChains.push_back(DAG.getStore(Chain, Arg, PtrOff, NULL, 0)); + if (!isTailCall) + MemOpChains.push_back(DAG.getStore(Chain, Arg, PtrOff, NULL, 0)); + // Calculate and remember argument location. + else + CalculateTailCallArgDest(DAG, MF, isPPC64, Arg, SPDiff, ArgOffset, + TailCallArguments); + inMem = true; } if (inMem || isMachoABI) { @@ -1994,7 +2339,13 @@ SDOperand PPCTargetLowering::LowerCALL(SDOperand Op, SelectionDAG &DAG, } }...

Hello Dale, this is an updated version of the tail call optimization patch for powerpc. could you have a look at it? i added code to support ppc64 (untested, will try to get access to ppc64 on a friend's machine). incorporated evan's formatting suggestions. ;) will run another round of testing (llvm-test) on my powerpc g4/800 when i get the okay to commit. testing on this machine takes

...t you mean by 'these' here. The function above IsEligibleForTailCallOptimization, IsPossiblyOverwrittenArgumentOfTailCall, etc. Basically anything that's shared between all the targets. > > (IsPossiblyOverwritten...) is used in > LowerCALL() > forall arguments > CalculateTailCallArgDest() > to determine whether an argument needs to be evacuated to a virtual > register to prevent being overwritten. > > I am not sure what you are suggesting ? Storing this information > (ispossiblyoverwritten) in the argument nodes? > as ARG_FLAGSSDNode as a new ArgFlagsTy? > &gt...