Sequence IL copy operand translation

[plafosse]

Summary

Evaluate multi-operand IL subexpressions into locals before constructing the copied expression.

Root cause

LowLevelILInstruction::CopyTo invoked a caller-provided subExprHandler multiple times as arguments to the same function call. C++ does not specify the order in which those arguments are evaluated. Stateful handlers, including the core LLIL-to-SSA translator's register-stack tracking callback, could therefore produce compiler-dependent output.

Impact

• preserves left-to-right operand translation for binary and carry operations
• preserves destination-before-source translation for stores and relative register-stack assignments
• makes callback side effects deterministic across MSVC and Clang

Validation

• compiled lowlevelilinstruction.cpp with the development build's C++20 flags using -fsyntax-only
• git diff --check passes

Companion core changes are in Vector35/binaryninja#1666.