assembly: use if instead of mul

use if success { ... }  instead of mul(success,....)

also, for EntryPointSimulations:
- had to reduce code size (so cleared simulation-unrelated
"supportsInterface")
- fixed maxgap

contracts/core/EntryPoint.sol

@@ -520,8 +520,11 @@ contract EntryPoint is IEntryPoint, StakeManager, NonceManager, ReentrancyGuardT
         uint256 dataSize;
         assembly ("memory-safe"){
             let success := call(gasLimit, sender, 0, add(callData, 0x20), mload(callData), 0, 32)
-            dataSize := mul(returndatasize(), success)
-            validationData := mload(0)
+            if success {
+                // ignore returndatasize, in case of revert
+                dataSize := returndatasize()
+                validationData := mload(0)
+            }
         }
         restoreFreePtr(saveFreePtr);
         if (dataSize != 32) {

contracts/core/EntryPointSimulations.sol

@@ -215,4 +215,7 @@ contract EntryPointSimulations is EntryPoint, IEntryPointSimulations {
         return __domainSeparatorV4;
     }
 
+    function supportsInterface(bytes4 interfaceId) public view virtual override returns (bool) {
+        return false;
+    }
 }

test/entrypointsimulations.test.ts

@@ -295,7 +295,7 @@ describe('EntryPointSimulations', function () {
       describe(`compare to execution ${withPaymaster} paymaster`, () => {
         let execVgl: number
         let execPmVgl: number
-        const diff = 500
+        const diff = 600
         before(async () => {
           execPmVgl = withPaymaster === 'without' ? 0 : await findUserOpWithMin(async n => userOpWithGas(1e6, n), false, entryPoint, 1, 500000)
           execVgl = await findUserOpWithMin(async n => userOpWithGas(n, execPmVgl), false, entryPoint, 1, 500000)