[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 mailing list