Skip to content
This repository has been archived by the owner on Oct 18, 2022. It is now read-only.

Rewrite the runtime bytecode and not just the constructor bytecode #17

Open
wants to merge 2 commits into
base: develop-0.7
Choose a base branch
from

Conversation

shellygr
Copy link
Contributor

@shellygr shellygr commented Mar 12, 2021

Description

The Optimism solc implementation is introducing a few tricks to avoid optimizer rewrites to the instrumented bytecode, and in a later pass searches and replaces the patterns with the real intended bytecode.
However, tools (like Certora's Prover tool) consume the runtime bytecode when they analyze contracts.
This PR applies the search-and-replace code to runtime bytecode as well.

Checklist

  • Code compiles correctly
  • All tests are passing
  • New tests have been created which fail without the change (if possible)
  • README / documentation was extended, if necessary
  • Changelog entry (if change is visible to the user)
  • Used meaningful commit messages

Fixes OP-833

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant