[cfe-commits] r139175 - in /cfe/trunk: lib/Sema/SemaDecl.cpp test/CodeGenObjCXX/nrvo.mm

Douglas Gregor dgregor at apple.com
Tue Sep 6 13:33:38 PDT 2011


Author: dgregor
Date: Tue Sep  6 15:33:37 2011
New Revision: 139175

URL: http://llvm.org/viewvc/llvm-project?rev=139175&view=rev
Log:
Implement the Named Return Value Optimization (NRVO) for Objective-C++
methods. Fixes PR10835 / <rdar://problem/10050178>.

Added:
    cfe/trunk/test/CodeGenObjCXX/nrvo.mm
Modified:
    cfe/trunk/lib/Sema/SemaDecl.cpp

Modified: cfe/trunk/lib/Sema/SemaDecl.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDecl.cpp?rev=139175&r1=139174&r2=139175&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaDecl.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDecl.cpp Tue Sep  6 15:33:37 2011
@@ -6700,6 +6700,9 @@
       DiagnoseUnusedParameters(MD->param_begin(), MD->param_end());
       DiagnoseSizeOfParametersAndReturnValue(MD->param_begin(), MD->param_end(),
                                              MD->getResultType(), MD);
+      
+      if (Body)
+        ComputeNRVO(Body, getCurFunction());
     }
     if (ObjCShouldCallSuperDealloc) {
       Diag(MD->getLocEnd(), diag::warn_objc_missing_super_dealloc);

Added: cfe/trunk/test/CodeGenObjCXX/nrvo.mm
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenObjCXX/nrvo.mm?rev=139175&view=auto
==============================================================================
--- cfe/trunk/test/CodeGenObjCXX/nrvo.mm (added)
+++ cfe/trunk/test/CodeGenObjCXX/nrvo.mm Tue Sep  6 15:33:37 2011
@@ -0,0 +1,21 @@
+// RUN: %clang_cc1 -emit-llvm -o - %s -O1 -triple x86_64-apple-darwin10.0.0 | FileCheck %s
+
+// PR10835 / <rdar://problem/10050178>
+struct X {
+  X();
+  X(const X&);
+  ~X();
+};
+
+ at interface NRVO
+ at end
+
+ at implementation NRVO
+// CHECK: define internal void @"\01-[NRVO getNRVO]"
+- (X)getNRVO { 
+  X x;
+  // CHECK: tail call void @_ZN1XC1Ev
+  // CHECK-NEXT: ret void
+  return x;
+}
+ at end





More information about the cfe-commits mailing list