Use invocation's interpreter

[turbolent]

Description

Noticed while implementing #2151: We are currently not consistently using the invocation's interpreter.

---

• Targeted PR against master branch
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work
• Code follows the standards mentioned here
• Updated relevant documentation
• Re-reviewed Files changed in the Github PR explorer
• Added appropriate labels

[codecov]

Codecov Report

Merging #2159 (d216869) into master (d35e886) will increase coverage by 0.11%.
The diff coverage is 100.00%.

❗ Current head d216869 differs from pull request most recent head 29c9723. Consider uploading reports for the commit 29c9723 to get more accurate results

@@            Coverage Diff             @@
##           master    #2159      +/-   ##
==========================================
+ Coverage   77.58%   77.69%   +0.11%     
==========================================
  Files         309      308       -1     
  Lines       64688    64272     -416     
==========================================
- Hits        50186    49936     -250     
+ Misses      12760    12600     -160     
+ Partials     1742     1736       -6     

Flag	Coverage Δ
unittests	77.69% <100.00%> (+0.11%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
runtime/interpreter/interpreter.go	89.26% <100.00%> (+0.09%)	⬆️
runtime/interpreter/value.go	68.98% <100.00%> (+0.15%)	⬆️
runtime/interpreter/errors.go	56.82% <0.00%> (-0.61%)	⬇️
runtime/sema/errors.go	91.88% <0.00%> (-0.23%)	⬇️
runtime/interpreter/account.go	92.66% <0.00%> (-0.20%)	⬇️
runtime/interpreter/decode.go	46.05% <0.00%> (-0.15%)	⬇️
runtime/stdlib/account.go	86.44% <0.00%> (-0.13%)	⬇️
runtime/interpreter/encode.go	67.60% <0.00%> (-0.11%)	⬇️
runtime/environment.go	91.73% <0.00%> (-0.09%)	⬇️
runtime/sema/checker.go	92.15% <0.00%> (-0.04%)	⬇️
... and 12 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[github-actions]

Cadence Benchstat comparison

This branch with compared with the base branch onflow:master commit d35e886
The command for i in {1..N}; do go test ./... -run=XXX -bench=. -benchmem -shuffle=on; done was used.
Bench tests were run a total of 7 times on each branch.

Collapsed results for better readability

	old.txt	new.txt
	time/op		delta
CheckContractInterfaceFungibleTokenConformance-2	116µs ± 5%	117µs ± 9%	~	(p=1.000 n=7+7)
ContractInterfaceFungibleToken-2	40.7µs ± 8%	39.7µs ± 3%	~	(p=0.101 n=7+6)
InterpretRecursionFib-2	2.34ms ± 8%	2.33ms ± 4%	~	(p=1.000 n=7+7)
NewInterpreter/new_interpreter-2	1.15µs ± 2%	1.17µs ± 4%	~	(p=0.167 n=6+6)
NewInterpreter/new_sub-interpreter-2	595ns ± 2%	599ns ± 5%	~	(p=0.652 n=6+7)
ParseArray-2	8.13ms ± 4%	8.15ms ± 8%	~	(p=0.805 n=7+7)
ParseDeploy/byte_array-2	12.2ms ± 3%	12.1ms ± 2%	~	(p=0.534 n=6+7)
ParseDeploy/decode_hex-2	1.27ms ± 5%	1.24ms ± 6%	~	(p=0.259 n=7+7)
ParseFungibleToken/With_memory_metering-2	190µs ± 4%	187µs ± 3%	~	(p=0.259 n=7+7)
ParseFungibleToken/Without_memory_metering-2	149µs ± 4%	147µs ± 4%	~	(p=0.318 n=7+7)
ParseInfix-2	6.86µs ± 2%	6.87µs ± 6%	~	(p=0.300 n=7+7)
QualifiedIdentifierCreation/One_level-2	2.34ns ± 3%	2.32ns ± 3%	~	(p=0.456 n=7+7)
QualifiedIdentifierCreation/Three_levels-2	142ns ± 4%	137ns ± 2%	−3.10%	(p=0.022 n=7+6)
RuntimeFungibleTokenTransfer-2	698µs ± 2%	570µs ±38%	~	(p=0.053 n=7+7)
RuntimeResourceDictionaryValues-2	5.38ms ± 5%	5.29ms ± 4%	~	(p=0.383 n=7+7)
RuntimeScriptNoop-2	25.0µs ± 6%	19.5µs ±36%	~	(p=0.234 n=6+7)
SuperTypeInference/arrays-2	324ns ± 6%	321ns ± 6%	~	(p=0.456 n=7+7)
SuperTypeInference/composites-2	130ns ± 4%	131ns ± 6%	~	(p=0.776 n=7+7)
SuperTypeInference/integers-2	91.2ns ± 4%	91.0ns ± 7%	~	(p=0.620 n=7+7)
ValueIsSubtypeOfSemaType-2	97.6ns ± 4%	95.2ns ± 3%	~	(p=0.128 n=7+7)
	alloc/op		delta
CheckContractInterfaceFungibleTokenConformance-2	49.8kB ± 0%	49.8kB ± 0%	~	(all equal)
ContractInterfaceFungibleToken-2	24.1kB ± 0%	24.1kB ± 0%	~	(all equal)
InterpretRecursionFib-2	1.00MB ± 0%	1.00MB ± 0%	~	(p=0.245 n=7+6)
NewInterpreter/new_interpreter-2	768B ± 0%	768B ± 0%	~	(all equal)
NewInterpreter/new_sub-interpreter-2	200B ± 0%	200B ± 0%	~	(all equal)
ParseArray-2	2.75MB ± 3%	2.74MB ± 5%	~	(p=0.805 n=7+7)
ParseDeploy/byte_array-2	4.17MB ± 3%	4.21MB ± 3%	~	(p=0.434 n=7+7)
ParseDeploy/decode_hex-2	214kB ± 0%	214kB ± 0%	~	(p=0.639 n=7+7)
ParseFungibleToken/With_memory_metering-2	29.2kB ± 0%	29.2kB ± 0%	~	(p=0.706 n=7+7)
ParseFungibleToken/Without_memory_metering-2	29.2kB ± 0%	29.2kB ± 0%	~	(p=0.270 n=7+7)
ParseInfix-2	1.91kB ± 0%	1.91kB ± 0%	~	(p=0.499 n=7+7)
QualifiedIdentifierCreation/One_level-2	0.00B	0.00B	~	(all equal)
QualifiedIdentifierCreation/Three_levels-2	64.0B ± 0%	64.0B ± 0%	~	(all equal)
RuntimeFungibleTokenTransfer-2	103kB ± 0%	102kB ± 1%	−1.03%	(p=0.011 n=7+7)
RuntimeResourceDictionaryValues-2	2.28MB ± 0%	2.28MB ± 0%	~	(p=0.902 n=7+7)
RuntimeScriptNoop-2	8.45kB ± 0%	8.44kB ± 0%	~	(p=0.535 n=7+7)
SuperTypeInference/arrays-2	96.0B ± 0%	96.0B ± 0%	~	(all equal)
SuperTypeInference/composites-2	0.00B	0.00B	~	(all equal)
SuperTypeInference/integers-2	0.00B	0.00B	~	(all equal)
ValueIsSubtypeOfSemaType-2	48.0B ± 0%	48.0B ± 0%	~	(all equal)
	allocs/op		delta
CheckContractInterfaceFungibleTokenConformance-2	815 ± 0%	815 ± 0%	~	(all equal)
ContractInterfaceFungibleToken-2	388 ± 0%	388 ± 0%	~	(all equal)
InterpretRecursionFib-2	18.9k ± 0%	18.9k ± 0%	~	(all equal)
NewInterpreter/new_interpreter-2	13.0 ± 0%	13.0 ± 0%	~	(all equal)
NewInterpreter/new_sub-interpreter-2	4.00 ± 0%	4.00 ± 0%	~	(all equal)
ParseArray-2	59.6k ± 0%	59.6k ± 0%	~	(p=0.286 n=7+7)
ParseDeploy/byte_array-2	89.4k ± 0%	89.4k ± 0%	~	(p=0.286 n=7+7)
ParseDeploy/decode_hex-2	64.0 ± 0%	64.0 ± 0%	~	(all equal)
ParseFungibleToken/With_memory_metering-2	779 ± 0%	779 ± 0%	~	(all equal)
ParseFungibleToken/Without_memory_metering-2	779 ± 0%	779 ± 0%	~	(all equal)
ParseInfix-2	48.0 ± 0%	48.0 ± 0%	~	(all equal)
QualifiedIdentifierCreation/One_level-2	0.00	0.00	~	(all equal)
QualifiedIdentifierCreation/Three_levels-2	2.00 ± 0%	2.00 ± 0%	~	(all equal)
RuntimeFungibleTokenTransfer-2	1.99k ± 0%	1.99k ± 0%	~	(all equal)
RuntimeResourceDictionaryValues-2	36.9k ± 0%	36.9k ± 0%	~	(p=0.906 n=7+7)
RuntimeScriptNoop-2	132 ± 0%	132 ± 0%	~	(all equal)
SuperTypeInference/arrays-2	3.00 ± 0%	3.00 ± 0%	~	(all equal)
SuperTypeInference/composites-2	0.00	0.00	~	(all equal)
SuperTypeInference/integers-2	0.00	0.00	~	(all equal)
ValueIsSubtypeOfSemaType-2	1.00 ± 0%	1.00 ± 0%	~	(all equal)