[clang] [Clang][Bytecode][NFC] Move Result into APSInt constructor (PR #153664)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 14 13:08:12 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Shafik Yaghmour (shafik)
<details>
<summary>Changes</summary>
Static analysis flagged this line because we are copying Result instead of moving it.
---
Full diff: https://github.com/llvm/llvm-project/pull/153664.diff
1 Files Affected:
- (modified) clang/lib/AST/ByteCode/InterpBuiltin.cpp (+1-1)
``````````diff
diff --git a/clang/lib/AST/ByteCode/InterpBuiltin.cpp b/clang/lib/AST/ByteCode/InterpBuiltin.cpp
index ee2d532551583..3c0dd8f901dc0 100644
--- a/clang/lib/AST/ByteCode/InterpBuiltin.cpp
+++ b/clang/lib/AST/ByteCode/InterpBuiltin.cpp
@@ -2285,7 +2285,7 @@ static bool interp__builtin_elementwise_sat(InterpState &S, CodePtr OpPC,
llvm_unreachable("Wrong builtin ID");
}
- pushInteger(S, APSInt(Result, !LHS.isSigned()), Call->getType());
+ pushInteger(S, APSInt(std::move(Result), !LHS.isSigned()), Call->getType());
return true;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/153664
More information about the cfe-commits
mailing list