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

[Vadzim Dambrouski via Phabricator via llvm-commits]

pftbest added a comment.

As I see it, CurrentArgIndex is overloaded with two (or more) tasks, so the code is confusing. Maybe it would be better if CurrentArgIndex was just a counter, and the other conditions can be handled by introducing boolean flags, for example.


https://reviews.llvm.org/D29069