[PATCH] D54583: PowerPC: Optimize SPE double parameter calling setup

Kei Thomsen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 18 03:00:23 PST 2019


kthomsen added a comment.

Hi Justin, I'm watching your work and used your patches to bring SPE into my CLANG for OS-9.
The OutVals[] issue is what I found yesterday as well by debugging the CLANG part.
There is a 2. location of this in PPCTargetLowering::LowerReturn()

   // Copy the result values into the output registers.
  for (unsigned i = 0, realI = 0; i != RVLocs.size(); ++i, ++realI) {
    CCValAssign &VA = RVLocs[i];
    assert(VA.isRegLoc() && "Can only return in registers!");
  
    SDValue Arg = OutVals[realI];

Regards, Kei


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54583/new/

https://reviews.llvm.org/D54583





More information about the llvm-commits mailing list