[PATCH] D47067: Update NRVO logic to support early return

Taiju Tsuiki via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 25 07:19:37 PDT 2018


tzik added inline comments.


================
Comment at: lib/Sema/Scope.cpp:128
 
-  if (getEntity())
-    return;
-
-  if (NRVO.getInt())
-    getParent()->setNoNRVO();
-  else if (NRVO.getPointer())
-    getParent()->addNRVOCandidate(NRVO.getPointer());
+  if (getParent())
+    getParent()->setNRVOCandidate(Candidate);
----------------
xbolva00 wrote:
> auto * Parent = getParent();
> if (Parent)
>     Parent>setNRVOCandidate(Candidate);
> 
> ?
Updated!


Repository:
  rC Clang

https://reviews.llvm.org/D47067





More information about the cfe-commits mailing list