[PATCH] D17294: Fix for PR 26500

[Nemanja Ivanovic via llvm-commits]

nemanjai updated this revision to Diff 48588.
nemanjai added a comment.

I think Kit mentioned that he still has some concerns about the semantics in the patch. In the meantime and in the interest of getting this patch finalized as soon as possible so that it stops blocking 3.8, I'm uploading an update that I feel addresses Hal's and Kit's comments.

The major change is that we don't set the output parameters to registers that aren't available. If any required register isn't available, we'll set the corresponding output parameter to PPC::NoRegister. This will ensure that we never try to use live registers as scratch registers. Also, to eliminate confusion about what the purpose of findScratchRegister is used for at any particular call, the return value is now always used. When we're emitting the prologue/epilogue, we use it to assert that we have enough registers.


Repository:
  rL LLVM

http://reviews.llvm.org/D17294

Files:
  lib/Target/PowerPC/PPCFrameLowering.cpp
  lib/Target/PowerPC/PPCFrameLowering.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17294.48588.patch
Type: text/x-patch
Size: 14775 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160220/06458ab1/attachment.bin>