From 325977c0c6947597618afca51494df62bacefb76 Mon Sep 17 00:00:00 2001 From: Nguyen Dac Nam Date: Fri, 13 Mar 2020 15:35:15 +0700 Subject: [PATCH] Apply suggestions from code review Co-Authored-By: Mat M. --- src/video_core/shader/decode/bfe.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/video_core/shader/decode/bfe.cpp b/src/video_core/shader/decode/bfe.cpp index 39587e0df9..7f2647973e 100644 --- a/src/video_core/shader/decode/bfe.cpp +++ b/src/video_core/shader/decode/bfe.cpp @@ -44,14 +44,14 @@ u32 ShaderIR::DecodeBfe(NodeBlock& bb, u32 pc) { Node v1 = SignedOperation(OperationCode::ILogicalShiftRight, is_signed, op_a, Immediate(s)); if (mask != 0) { - v1 = SignedOperation(OperationCode::IBitwiseAnd, is_signed, v1, Immediate(mask)); + v1 = SignedOperation(OperationCode::IBitwiseAnd, is_signed, std::move(v1), Immediate(mask)); } Node v2 = op_a; if (mask != 0) { - v2 = SignedOperation(OperationCode::IBitwiseAnd, is_signed, op_a, Immediate(mask)); + v2 = SignedOperation(OperationCode::IBitwiseAnd, is_signed, std::move(v2), Immediate(mask)); } - v2 = SignedOperation(OperationCode::ILogicalShiftLeft, is_signed, v2, Immediate(s)); - return SignedOperation(OperationCode::IBitwiseOr, is_signed, v1, v2); + v2 = SignedOperation(OperationCode::ILogicalShiftLeft, is_signed, std::move(v2), Immediate(s)); + return SignedOperation(OperationCode::IBitwiseOr, is_signed, std::move(v1), std::move(v2)); }; op_a = swap(1, 0x55555555U); op_a = swap(2, 0x33333333U); @@ -66,7 +66,7 @@ u32 ShaderIR::DecodeBfe(NodeBlock& bb, u32 pc) { Immediate(8), Immediate(8)); const auto result = SignedOperation(OperationCode::IBitfieldExtract, is_signed, op_a, offset, bits); - SetRegister(bb, instr.gpr0, result); + SetRegister(bb, instr.gpr0, std::move(result)); return pc; }