[PATCH] D29069: [MSP430] Add SRet support to MSP430 target
Andrew Wygle via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Feb 4 15:20:57 PST 2017
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.
More information about the llvm-commits