[PATCH] D20311: [PowerPC] Add a testcase for TCO on string rvo function

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Mon May 23 12:58:55 PDT 2016


echristo added a comment.

I know you've already committed, but a couple of small requests.

Thanks!


================
Comment at: llvm/trunk/test/CodeGen/PowerPC/tailcall-string-rvo.ll:19-23
@@ +18,7 @@
+
+; string TestFoo() {
+;   string ret = undef;
+;   TestBar(&ret);  // tail call optimized
+;   return ret;
+; }
+define void @TestFoo(%class.basic_string.11.42.73* noalias sret %arg) {
----------------
Can you move this to the top please? And add a comment of what you want to be testing here?


Repository:
  rL LLVM

http://reviews.llvm.org/D20311





More information about the llvm-commits mailing list