status-im/dagger-contracts
2
0
Fork 0
mirror of https://github.com/status-im/dagger-contracts.git synced 2025-02-26 13:25:22 +00:00
Code Issues Projects Releases Wiki Activity

Remove calls to invalid()

Browse Source 
Gas estimation seems to work just fine without them?
This commit is contained in:
Mark Spanbroek 2024-01-30 09:09:59 +01:00 committed by markspanbroek
parent c3d7db3456
commit 80dfa41e32
1 changed files with 0 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
contracts/Groth16Verifier.sol
View File

@ -45,11 +45,6 @@ library Pairing {
// solhint-disable-next-line no-inline-assembly // solhint-disable-next-line no-inline-assembly
assembly { assembly {
success := staticcall(sub(gas(), 2000), 6, input, 0xc0, r, 0x60) success := staticcall(sub(gas(), 2000), 6, input, 0xc0, r, 0x60)
// Use "invalid" to make gas estimation work
switch success
case 0 {
invalid()
}
} }
require(success, "pairing-add-failed"); require(success, "pairing-add-failed");
} }
@ -68,11 +63,6 @@ library Pairing {
// solhint-disable-next-line no-inline-assembly // solhint-disable-next-line no-inline-assembly
assembly { assembly {
success := staticcall(sub(gas(), 2000), 7, input, 0x80, r, 0x60) success := staticcall(sub(gas(), 2000), 7, input, 0x80, r, 0x60)
// Use "invalid" to make gas estimation work
switch success
case 0 {
invalid()
}
} }
require(success, "pairing-mul-failed"); require(success, "pairing-mul-failed");
} }
@ -109,11 +99,6 @@ library Pairing {
out, out,
0x20 0x20
) )
// Use "invalid" to make gas estimation work
switch success
case 0 {
invalid()
}
} }
require(success, "pairing-opcode-failed"); require(success, "pairing-opcode-failed");
return out[0] != 0; return out[0] != 0;