[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