search for: ispossiblyoverwritten

...leForTailCallOptimization. Is it possible > to make it into a common utility function (or at least the target > independent part)? The target independent part can be refactored. The whole function not because e.g ppc does not support yet fully support PIC. > > Also > +static bool IsPossiblyOverwrittenArgumentOfTailCall(SDOperand Op, > + MachineFrameInfo > * 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...

...independent part)? > The target independent part can be refactored. The whole function not > because e.g ppc does not support yet fully support PIC. That's fine. Please break it into two parts and move the target independent part out. > >> >> Also >> +static bool IsPossiblyOverwrittenArgumentOfTailCall(SDOperand Op, >> + MachineFrameInfo >> * 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...

...es()-1) && + Ret.getOperand(1) == SDOperand(Call.Val,0))) { ... This function seems to be very similar to X86TargetLowering::IsEligibleForTailCallOptimization. Is it possible to make it into a common utility function (or at least the target independent part)? Also +static bool IsPossiblyOverwrittenArgumentOfTailCall(SDOperand Op, + MachineFrameInfo * MFI) { I wonder if these can be moved to SelectionDAGISel? So it's handled during the sdisel phase of call lowering? if (isPPC64 && Arg.getValueType() == MVT::i32) {...

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