[PATCH] D119927: [Clang] [P2025] More exhaustive tests for NRVO

Evgeny Shulgin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 16 12:47:25 PST 2022


Izaron updated this revision to Diff 409374.
Izaron added a comment.

Thanks! Yes I should've write comments that are understandable not only for me =)
I added comments to the existing tests as well.
Though NRVO attribute is bound to the variable, I'm also more comfortable to place comments in the "return" lines, as it is looks somewhat more clear.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119927/new/

https://reviews.llvm.org/D119927

Files:
  clang/test/CodeGenCXX/nrvo.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D119927.409374.patch
Type: text/x-patch
Size: 78550 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220216/9e0d0097/attachment-0001.bin>


More information about the cfe-commits mailing list