Properly transfer indexing expression

[turbolent]

Port of #1565 to master.

There were conflicts, so I didn't push directly but opted for a safer PR

---

• 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

[github-actions]

Cadence Benchstat comparison

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

Results

	old.txt	new.txt
	time/op		delta
RuntimeResourceDictionaryValues-2	16.3ms ± 6%	16.8ms ± 4%	~	(p=0.165 n=7+7)
RuntimeFungibleTokenTransfer-2	1.55ms ±23%	1.39ms ±36%	~	(p=0.620 n=7+7)
ParseFungibleToken-2	224µs ± 6%	227µs ± 2%	~	(p=0.620 n=7+7)
ParseDeploy/byte_array-2	22.6ms ± 6%	23.3ms ± 3%	~	(p=0.209 n=7+7)
ParseDeploy/decode_hex-2	1.42ms ± 5%	1.47ms ± 3%	~	(p=0.165 n=7+7)
ParseArray-2	14.6ms ± 9%	14.6ms ± 6%	~	(p=0.805 n=7+7)
ParseInfix-2	9.78µs ± 9%	9.94µs ± 3%	~	(p=0.318 n=7+7)
QualifiedIdentifierCreation/One_level-2	3.30ns ± 6%	3.27ns ± 5%	~	(p=0.902 n=7+7)
QualifiedIdentifierCreation/Three_levels-2	170ns ± 4%	173ns ± 7%	~	(p=0.456 n=7+7)
CheckContractInterfaceFungibleTokenConformance-2	169µs ± 7%	173µs ± 4%	~	(p=0.318 n=7+7)
ContractInterfaceFungibleToken-2	46.7µs ± 8%	48.3µs ± 3%	~	(p=0.053 n=7+7)
InterpretRecursionFib-2	3.13ms ± 9%	3.25ms ± 2%	~	(p=0.101 n=7+6)
NewInterpreter/new_interpreter-2	1.36µs ± 3%	1.33µs ± 5%	~	(p=0.295 n=6+7)
NewInterpreter/new_sub-interpreter-2	2.61µs ± 9%	2.61µs ± 7%	~	(p=0.902 n=7+7)
	alloc/op		delta
ContractInterfaceFungibleToken-2	26.6kB ± 0%	26.7kB ± 0%	+0.36%	(p=0.001 n=6+7)
CheckContractInterfaceFungibleTokenConformance-2	66.2kB ± 0%	66.3kB ± 0%	+0.15%	(p=0.001 n=7+7)
RuntimeFungibleTokenTransfer-2	273kB ± 0%	273kB ± 0%	+0.03%	(p=0.001 n=7+7)
RuntimeResourceDictionaryValues-2	4.05MB ± 0%	4.05MB ± 0%	+0.01%	(p=0.017 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)
InterpretRecursionFib-2	1.26MB ± 0%	1.26MB ± 0%	~	(p=0.450 n=7+6)
NewInterpreter/new_interpreter-2	848B ± 0%	848B ± 0%	~	(all equal)
NewInterpreter/new_sub-interpreter-2	1.34kB ± 0%	1.34kB ± 0%	~	(all equal)
	allocs/op		delta
ContractInterfaceFungibleToken-2	458 ± 0%	460 ± 0%	+0.44%	(p=0.001 n=7+7)
CheckContractInterfaceFungibleTokenConformance-2	1.07k ± 0%	1.07k ± 0%	+0.19%	(p=0.001 n=7+7)
RuntimeFungibleTokenTransfer-2	4.54k ± 0%	4.54k ± 0%	+0.05%	(p=0.008 n=7+7)
RuntimeResourceDictionaryValues-2	102k ± 0%	102k ± 0%	+0.00%	(p=0.001 n=7+7)
QualifiedIdentifierCreation/One_level-2	0.00	0.00	~	(all equal)
QualifiedIdentifierCreation/Three_levels-2	2.00 ± 0%	2.00 ± 0%	~	(all equal)
InterpretRecursionFib-2	26.2k ± 0%	26.2k ± 0%	~	(all equal)
NewInterpreter/new_interpreter-2	13.0 ± 0%	13.0 ± 0%	~	(all equal)
NewInterpreter/new_sub-interpreter-2	40.0 ± 0%	40.0 ± 0%	~	(all equal)