Add unstable-api feature flag to pallet-revive

substrate/frame/revive/Cargo.toml

@@ -75,6 +75,10 @@ xcm-builder = { workspace = true, default-features = true }
 
 [features]
 default = ["std"]
+unstable-api = [
+	"pallet-revive-uapi/unstable-api",
+	"pallet-revive-fixtures/unstable-api",
+]
 std = [
 	"codec/std",
 	"environmental/std",

substrate/frame/revive/fixtures/Cargo.toml

@@ -29,5 +29,6 @@ anyhow = { workspace = true, default-features = true }
 
 [features]
 default = ["std"]
+unstable-api = []
 # only when std is enabled all fixtures are available
 std = ["anyhow", "frame-system", "log/std", "sp-core", "sp-io", "sp-runtime"]

substrate/frame/revive/fixtures/build/_Cargo.toml

@@ -16,6 +16,9 @@ uapi = { package = 'pallet-revive-uapi', path = "", default-features = false }
 common = { package = 'pallet-revive-fixtures-common', path = "" }
 polkavm-derive = { version = "0.17.0" }
 
+[features]
+unstable-api = []
+
 [profile.release]
 opt-level = 3
 lto = true

substrate/frame/revive/fixtures/contracts/block_hash.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/call_diverging_out_len.rs

@@ -25,6 +25,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 

substrate/frame/revive/fixtures/contracts/call_return_code.rs

@@ -20,6 +20,7 @@
 //! It also forwards its input to the callee.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/call_runtime.rs

@@ -18,6 +18,7 @@
 //! This passes its input to `call_runtime` and returns the return value to its caller.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/caller_is_origin_n.rs

@@ -19,6 +19,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/chain_extension.rs

@@ -18,6 +18,7 @@
 //! Call chain extension by passing through input and output of this contract.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/chain_extension_temp_storage.rs

@@ -19,6 +19,7 @@
 //! It then calls itself once
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/code_hash.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/common/Cargo.toml

@@ -8,4 +8,4 @@ license.workspace = true
 description = "Common utilities for pallet-revive-fixtures."
 
 [dependencies]
-uapi = { package = 'pallet-revive-uapi', path = "../../../uapi", default-features = false }
+uapi = { package = 'pallet-revive-uapi', path = "../../../uapi", default-features = false }

substrate/frame/revive/fixtures/contracts/create_storage_and_call.rs

@@ -18,6 +18,7 @@
 //! This calls another contract as passed as its account id. It also creates some storage.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api, StorageFlags};

substrate/frame/revive/fixtures/contracts/create_storage_and_instantiate.rs

@@ -18,6 +18,7 @@
 //! This instantiates another contract and passes some input to its constructor.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::{input, u256_bytes};
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/crypto_hashes.rs

@@ -61,7 +61,9 @@ pub extern "C" fn call() {
 	match chosen_hash_fn {
 		1 => {
 			let mut output = [0u8; 32];
+			#[cfg(feature = "unstable-api")]
 			api::hash_sha2_256(input, &mut output);
+			#[cfg(feature = "unstable-api")]
 			api::return_value(uapi::ReturnFlags::empty(), &output);
 		},
 		2 => {
@@ -71,12 +73,16 @@ pub extern "C" fn call() {
 		},
 		3 => {
 			let mut output = [0u8; 32];
+			#[cfg(feature = "unstable-api")]
 			api::hash_blake2_256(input, &mut output);
+			#[cfg(feature = "unstable-api")]
 			api::return_value(uapi::ReturnFlags::empty(), &output);
 		},
 		4 => {
 			let mut output = [0u8; 16];
+			#[cfg(feature = "unstable-api")]
 			api::hash_blake2_128(input, &mut output);
+			#[cfg(feature = "unstable-api")]
 			api::return_value(uapi::ReturnFlags::empty(), &output);
 		},
 		_ => panic!("unknown crypto hash function identifier"),

substrate/frame/revive/fixtures/contracts/debug_message_invalid_utf8.rs

@@ -18,6 +18,7 @@
 //! Emit a debug message with an invalid utf-8 code.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/debug_message_logging_disabled.rs

@@ -28,6 +28,7 @@ pub extern "C" fn deploy() {}
 
 #[no_mangle]
 #[polkavm_derive::polkavm_export]
+#[cfg(feature = "unstable-api")]
 pub extern "C" fn call() {
 	assert_eq!(api::debug_message(b"Hello World!"), Err(ReturnErrorCode::LoggingDisabled));
 }

substrate/frame/revive/fixtures/contracts/debug_message_works.rs

@@ -18,6 +18,7 @@
 //! Emit a "Hello World!" debug message.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/drain.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::{u256_bytes, u64_output};
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/dummy.rs

@@ -16,6 +16,7 @@
 // limitations under the License.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 

substrate/frame/revive/fixtures/contracts/ecdsa_recover.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/event_and_return_on_deploy.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/instantiate_return_code.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::{input, u256_bytes};
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/instr_benchmark.rs

@@ -16,6 +16,7 @@
 // limitations under the License.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 

substrate/frame/revive/fixtures/contracts/locking_delegate_dependency.rs

@@ -19,6 +19,7 @@
 //! calling into a contract.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/new_set_code_hash_contract.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/ok_trap_revert.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/oom_ro.rs

@@ -21,6 +21,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 

substrate/frame/revive/fixtures/contracts/oom_rw_included.rs

@@ -21,6 +21,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 

substrate/frame/revive/fixtures/contracts/oom_rw_trailing.rs

@@ -21,6 +21,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 

substrate/frame/revive/fixtures/contracts/return_with_data.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api, StorageFlags};
@@ -40,6 +41,7 @@ pub extern "C" fn call() {
 
 	// Burn some PoV, clear_storage consumes some PoV as in order to clear the storage we need to we
 	// need to read its size first.
+	#[cfg(feature = "unstable-api")]
 	api::clear_storage(StorageFlags::empty(), b"");
 
 	let exit_status = uapi::ReturnFlags::from_bits(exit_status[0] as u32).unwrap();

substrate/frame/revive/fixtures/contracts/self_destructing_constructor.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/set_code_hash.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/sr25519_verify.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/storage.rs

@@ -19,6 +19,7 @@
 //! versions of the storage APIs.
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::unwrap_output;
 use uapi::{HostFn, HostFnImpl as api, StorageFlags};

substrate/frame/revive/fixtures/contracts/terminate_and_send_to_eve.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 extern crate common;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/transfer_return_code.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::u256_bytes;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/xcm_execute.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};

substrate/frame/revive/fixtures/contracts/xcm_send.rs

@@ -17,6 +17,7 @@
 
 #![no_std]
 #![no_main]
+#![cfg(feature = "unstable-api")]
 
 use common::input;
 use uapi::{HostFn, HostFnImpl as api};