From 73825e3f8d2e3a680ca959717f70c36d0cefe231 Mon Sep 17 00:00:00 2001 From: Joe Caulfield Date: Thu, 19 Dec 2024 18:08:11 +0800 Subject: [PATCH] builtins: update contributor comments --- builtins/src/lib.rs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/builtins/src/lib.rs b/builtins/src/lib.rs index 05a4cf36515703..02a437db70cf01 100644 --- a/builtins/src/lib.rs +++ b/builtins/src/lib.rs @@ -46,11 +46,11 @@ macro_rules! testable_prototype { }; } -/// DEVELOPER: when a builtin is migrated to sbpf, please add its corresponding -/// migration feature ID to solana-builtin-default-costs::BUILTIN_INSTRUCTION_COSTS, -/// so the builtin's default cost can be determined properly based on feature status. -/// When migration completed, and the feature gate is enabled everywhere, please -/// remove that builtin entry from solana-builtin-default-costs::BUILTIN_INSTRUCTION_COSTS. +/// CONTRIBUTOR: When a builtin is scheduled to be migrated to Core BPF, add +/// the `CoreBpfMigrationConfig` to the builtin prototype here, and adjust the +/// length value in `cost_modeling::NUM_COST_MODELED_BUILTINS_WITH_MIGRATIONS`. +/// When the migration is completed and the feature is cleaned up, remove the +/// builtin from this list and update the length value again. pub static BUILTINS: &[BuiltinPrototype] = &[ testable_prototype!(BuiltinPrototype { name: system_program,