Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport SignextLowering #1189 + #1280 #1286

Merged
merged 8 commits into from
Aug 22, 2023
Merged

Backport SignextLowering #1189 + #1280 #1286

merged 8 commits into from
Aug 22, 2023

Conversation

ascjones
Copy link
Collaborator

@ascjones ascjones commented Aug 21, 2023

Backport #1189 + #1280. Enables new toolchains > 1.69 + nodes with old pallet_contracts (pre sign_ext support)

Edit: Also had to manually backport subxt updates from #1149 and #1236 in order to fix compatibility issues with an old version of sp-core on which the current version of subxt depends. wasmi-validation does not expect the sign-ext instruction, but this dependency is removed in recent releases.

Can be released as 3.2.0. Rel #1239

ascjones and others added 3 commits August 21, 2023 12:38
* Add `SignextLowering` pass to `wasm-opt`

* Remove check for Rust version

(cherry picked from commit 756aa38)
* Enable Wasm sign_ext

* Bump substrate crates

* Add warning when building with new rustc

* Stable toolchain

* Revert "Stable toolchain"

This reverts commit aa2a2e6.

---------

Co-authored-by: Andrew Jones <[email protected]>
@ascjones ascjones changed the title Backport SignextLowering #1280 Backport SignextLowering #1189 + #1280 Aug 21, 2023
@SkymanOne
Copy link
Contributor

Is CI supposed to be failing?

@ascjones
Copy link
Collaborator Author

Is CI supposed to be failing?

No, I need to fix it.

@ascjones ascjones merged commit 8a8f0cb into v3.x.x Aug 22, 2023
10 checks passed
@ascjones ascjones deleted the aj/backport-signext branch August 22, 2023 08:58
@ascjones ascjones mentioned this pull request Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants