core/vm: fix comment and remove unused divisor in osakaModexpGas (#32553)

zach871 · web-flow · commit b708feb9c34b · 2025-09-10T21:44:18.000+08:00
The `divisor` const is not needed in the gas cost calculation in `osakaModexpGas`.
diff --git a/core/vm/contracts.go b/core/vm/contracts.go
@@ -538,7 +538,6 @@ func berlinModexpGas(baseLen, expLen, modLen uint64, expHead uint256.Int) uint64
 func osakaModexpGas(baseLen, expLen, modLen uint64, expHead uint256.Int) uint64 {
 	const (
 		multiplier = 16
-		divisor    = 3
 		minGas     = 500
 	)
 
@@ -549,7 +548,7 @@ func osakaModexpGas(baseLen, expLen, modLen uint64, expHead uint256.Int) uint64
 	}
 	iterationCount := modexpIterationCount(expLen, expHead, multiplier)
 
-	// Calculate gas: (multComplexity * iterationCount) / osakaDivisor
+	// Calculate gas: multComplexity * iterationCount
 	carry, gas := bits.Mul64(iterationCount, multComplexity)
 	if carry != 0 {
 		return math.MaxUint64

Original file line number	Diff line number	Diff line change
`@@ -538,7 +538,6 @@ func berlinModexpGas(baseLen, expLen, modLen uint64, expHead uint256.Int) uint64`
`538`	`538`	`func osakaModexpGas(baseLen, expLen, modLen uint64, expHead uint256.Int) uint64 {`
`539`	`539`	`const (`
`540`	`540`	`multiplier = 16`
`541`		`- divisor = 3`
`542`	`541`	`minGas = 500`
`543`	`542`	`)`
`544`	`543`
`@@ -549,7 +548,7 @@ func osakaModexpGas(baseLen, expLen, modLen uint64, expHead uint256.Int) uint64`
`549`	`548`	`}`
`550`	`549`	`iterationCount := modexpIterationCount(expLen, expHead, multiplier)`
`551`	`550`
`552`		`- // Calculate gas: (multComplexity * iterationCount) / osakaDivisor`
	`551`	`+ // Calculate gas: multComplexity * iterationCount`
`553`	`552`	`carry, gas := bits.Mul64(iterationCount, multComplexity)`
`554`	`553`	`if carry != 0 {`
`555`	`554`	`return math.MaxUint64`