Skip to content

Commit 6077ecb

Browse files
aaronbuchwaldgzliudan
authored andcommitted
core/vm: replace repeated string with variable in tests (ethereum#22774)
1 parent 446b9e8 commit 6077ecb

File tree

1 file changed

+35
-34
lines changed

1 file changed

+35
-34
lines changed

core/vm/instructions_test.go

Lines changed: 35 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ type twoOperandParams struct {
4141
y string
4242
}
4343

44+
var alphabetSoup = "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
4445
var commonParams []*twoOperandParams
4546
var twoOpMethods map[string]executionFunc
4647

@@ -354,8 +355,8 @@ func BenchmarkOpSub256(b *testing.B) {
354355
}
355356

356357
func BenchmarkOpMul(b *testing.B) {
357-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
358-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
358+
x := alphabetSoup
359+
y := alphabetSoup
359360

360361
opBenchmark(b, opMul, x, y)
361362
}
@@ -386,64 +387,64 @@ func BenchmarkOpSdiv(b *testing.B) {
386387
}
387388

388389
func BenchmarkOpMod(b *testing.B) {
389-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
390-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
390+
x := alphabetSoup
391+
y := alphabetSoup
391392

392393
opBenchmark(b, opMod, x, y)
393394
}
394395

395396
func BenchmarkOpSmod(b *testing.B) {
396-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
397-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
397+
x := alphabetSoup
398+
y := alphabetSoup
398399

399400
opBenchmark(b, opSmod, x, y)
400401
}
401402

402403
func BenchmarkOpExp(b *testing.B) {
403-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
404-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
404+
x := alphabetSoup
405+
y := alphabetSoup
405406

406407
opBenchmark(b, opExp, x, y)
407408
}
408409

409410
func BenchmarkOpSignExtend(b *testing.B) {
410-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
411-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
411+
x := alphabetSoup
412+
y := alphabetSoup
412413

413414
opBenchmark(b, opSignExtend, x, y)
414415
}
415416

416417
func BenchmarkOpLt(b *testing.B) {
417-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
418-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
418+
x := alphabetSoup
419+
y := alphabetSoup
419420

420421
opBenchmark(b, opLt, x, y)
421422
}
422423

423424
func BenchmarkOpGt(b *testing.B) {
424-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
425-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
425+
x := alphabetSoup
426+
y := alphabetSoup
426427

427428
opBenchmark(b, opGt, x, y)
428429
}
429430

430431
func BenchmarkOpSlt(b *testing.B) {
431-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
432-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
432+
x := alphabetSoup
433+
y := alphabetSoup
433434

434435
opBenchmark(b, opSlt, x, y)
435436
}
436437

437438
func BenchmarkOpSgt(b *testing.B) {
438-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
439-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
439+
x := alphabetSoup
440+
y := alphabetSoup
440441

441442
opBenchmark(b, opSgt, x, y)
442443
}
443444

444445
func BenchmarkOpEq(b *testing.B) {
445-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
446-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
446+
x := alphabetSoup
447+
y := alphabetSoup
447448

448449
opBenchmark(b, opEq, x, y)
449450
}
@@ -453,45 +454,45 @@ func BenchmarkOpEq2(b *testing.B) {
453454
opBenchmark(b, opEq, x, y)
454455
}
455456
func BenchmarkOpAnd(b *testing.B) {
456-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
457-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
457+
x := alphabetSoup
458+
y := alphabetSoup
458459

459460
opBenchmark(b, opAnd, x, y)
460461
}
461462

462463
func BenchmarkOpOr(b *testing.B) {
463-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
464-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
464+
x := alphabetSoup
465+
y := alphabetSoup
465466

466467
opBenchmark(b, opOr, x, y)
467468
}
468469

469470
func BenchmarkOpXor(b *testing.B) {
470-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
471-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
471+
x := alphabetSoup
472+
y := alphabetSoup
472473

473474
opBenchmark(b, opXor, x, y)
474475
}
475476

476477
func BenchmarkOpByte(b *testing.B) {
477-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
478-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
478+
x := alphabetSoup
479+
y := alphabetSoup
479480

480481
opBenchmark(b, opByte, x, y)
481482
}
482483

483484
func BenchmarkOpAddmod(b *testing.B) {
484-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
485-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
486-
z := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
485+
x := alphabetSoup
486+
y := alphabetSoup
487+
z := alphabetSoup
487488

488489
opBenchmark(b, opAddmod, x, y, z)
489490
}
490491

491492
func BenchmarkOpMulmod(b *testing.B) {
492-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
493-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
494-
z := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
493+
x := alphabetSoup
494+
y := alphabetSoup
495+
z := alphabetSoup
495496

496497
opBenchmark(b, opMulmod, x, y, z)
497498
}

0 commit comments

Comments
 (0)