[PATCH] D28388: Add ArgumentCopyElision MI pass

[Bob Haarman via Phabricator via llvm-commits]

inglorion requested changes to this revision.
inglorion added a comment.
This revision now requires changes to proceed.

The implementation looks good to me. I think the ARM test is missing a few cases.



================
Comment at: test/CodeGen/ARM/argument-copy-elision.ll:5
+
+declare i8* @llvm.returnaddress(i32)
+
----------------
I expect this should have the retaddr_taken and size_mismatch tests, too. If not, I think you can remove this declaration.


https://reviews.llvm.org/D28388