Solid Pull Request #1
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[SMTChecker] Fix comparison of `CHCVerificationTarget` for proper `std::set` ordering
SSA-CFG: Tiny code simplifications
Add Cyfrin's glossary to Resources in the docs
…ar-const refactor(formal): remove unused `clear()`/`solver()` from `EncodingContext`
In some circumstances (in particular if not the AST is not in SSA form), it could happen that the ExpressionSimplifier would substitute out-of-scope variables.
…on_simplifier Yul expression simplifier: Don't substitute out of scope variables
(cherry picked from commit 9697efe)
…ersion Pin hardhat to last 2.x.x version
…l-tests Bump node image for external tests running on hardhat
Boost.System has been header-only since Boost 1.69 and will be dropping the compatibility stub library in Boost 1.89 (boostorg/system@7a495bb). Since the `system` component is only needed for Boost < 1.69, it can be added to `BOOST_COMPONENTS` in the fallback find_package. Signed-off-by: Michael Cho <michael@michaelcho.dev>
Remove the hard dependency on `boost::system` compatibility library to fix builds with Boost 1.89.0
…-function-behavior fix: correct fromHex function documentation
update to boost 1.70.0 for non-windows builds
Update links to github org as argotorg and ipsilon where appropriate
…missions Update permissions for welcome-external-pr bot commenting
Fix external tests for hardhat-gas-reporter v2 compatibility
Co-authored-by: Kamil Śliwak <cameel2@gmail.com>
ci: Replace `t_ubu_soltest_all` with `t_ubu_soltest`
CI: Batched antlr check
Add Slippy to the list of Solidity tools
Remove unused errs assignment in jsonParseStrict success path
…i_parser CLI Parser: Use defaulted comparison operators for options
…d ahead of time and only merged after release
…-update Post-0.8.33 release checklist update
Replace `transfer` (deprecated) occurrences with `call` in smt tests
…idity Mention unofficial Nix builds in Installing Solidity
…s-from-default-sequence Remove redundant prerequisite steps from the default optimizer sequence
…utable Theoretically, loadimmutable inserts a placeholder into the contract to be deployed which is then replaced by setimmutable once the runtime code is loaded into memory. The interpreter is mainly used for differential fuzzing so we should satisfy that correct code never fails and we can still detect bugs otherwise, most of the time. Therefore loadimmutable is decoupled from setimmutable and just returns a random value (hash of the identifier), setimmutable is treated as a no-op.
…iss_immutables EVM instruction interpreter: Add loadimmutable and setimmutable mocks, improve linkersymbol
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
18 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
solidity pr