[PATCH] D29069: [MSP430] Add SRet support to MSP430 target

[Andrew Wygle via Phabricator via llvm-commits]

awygle added a comment.

The problem with ParseFunctionArgs was that it was being too clever with initializing CurrentArgIndex. It used to initialize to UINT16_MAX because that was "one less than" the first Arg's OrigArgIndex, but now that we have srets we can get NoArgIndex which happens to be UINT16_MAX. I originally tried to preserve the structure of the loop but now it just explicitly initializes instead of being clever.


https://reviews.llvm.org/D29069