sui icon indicating copy to clipboard operation
sui copied to clipboard

[move][bug] Fix binop ordering

Open cgswords opened this issue 10 months ago • 1 comments

Description

Fixes an issue in how binops evaluated their arguments, ensuring the LHS value is stored if the RHS has effect operations.

Test plan

Tests thanks to Aptos: https://github.com/aptos-labs/aptos-core/pull/12936


Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

  • [ ] Protocol:
  • [ ] Nodes (Validators and Full nodes):
  • [ ] Indexer:
  • [ ] JSON-RPC:
  • [ ] GraphQL:
  • [ ] CLI:
  • [ ] Rust SDK:

cgswords avatar Apr 25 '24 20:04 cgswords

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sui-core ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 29, 2024 5:54pm
sui-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 29, 2024 5:54pm
3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Apr 29, 2024 5:54pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Apr 29, 2024 5:54pm
sui-typescript-docs ⬜️ Ignored (Inspect) Visit Preview Apr 29, 2024 5:54pm

vercel[bot] avatar Apr 25 '24 20:04 vercel[bot]