diff options
author | Grigory Pastukhov <99913765+grigorypas@users.noreply.github.com> | 2025-05-16 11:47:53 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-05-16 15:47:53 -0300 |
commit | 48587f30d63689816f3d5a1d199dba42ff332247 (patch) | |
tree | dbdbade5aa010c5426fafa637e05e6d77473dd2d /clang/lib/Sema/SemaDecl.cpp | |
parent | 4e01a07cffc43f720ed527969ba70c396940ce2e (diff) | |
download | llvm-48587f30d63689816f3d5a1d199dba42ff332247.zip llvm-48587f30d63689816f3d5a1d199dba42ff332247.tar.gz llvm-48587f30d63689816f3d5a1d199dba42ff332247.tar.bz2 |
[clang] Add new warning: not eliding copy on return (missed NRVO) (#139973)
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
-rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index a7d59ec2..6dae243 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -16093,8 +16093,11 @@ void Sema::computeNRVO(Stmt *Body, FunctionScopeInfo *Scope) { for (unsigned I = 0, E = Scope->Returns.size(); I != E; ++I) { if (const VarDecl *NRVOCandidate = Returns[I]->getNRVOCandidate()) { - if (!NRVOCandidate->isNRVOVariable()) + if (!NRVOCandidate->isNRVOVariable()) { + Diag(Returns[I]->getRetValue()->getExprLoc(), + diag::warn_not_eliding_copy_on_return); Returns[I]->setNRVOCandidate(nullptr); + } } } } |