dxo
dxo
Maybe it’s better to just profile `hevm bc-test` directly?
results of running `cabal new-run hevm -- bc-test --file ~/code/github.com/ethereum/tests/BlockchainTests/GeneralStateTests/stQuadraticComplexityTest/Return50000.json --timeout 99999999 +RTS -p`: https://ipfs.io/ipfs/QmcMTQ6jEHk65xnGFHVvxxjCgtvwSxPaFDmRatn1D8hkCP/hevm.prof This time we're getting 40% of the alloc from `byteStringSliceWithDefaultZeroes` in `Concrete.hs`
So I made `byteStringSliceWithDefaultZeroes` as strict as I know how (see [here](https://github.com/dapphub/dapptools/compare/mem-usage)), and the memory usage didn't seem to change anything (profiling output [here](https://ipfs.io/ipfs/QmWDB145QCQ5iqNiC3nXfvfPB9yAMgfyDLEMnyUQFbUXrJ/hevm.prof)). Not really sure how to proceed.
@lucas-manuel I think that the above procedure may break bytecode verification since edits to the source files are reflected in the metadata hash appended to the resulting btyecode. It's also...
I think we could modify `dapp rempapings` to allow multiple verisons of a given package by using the context feature of the remappings format ([docs](https://docs.soliditylang.org/en/v0.8.6/path-resolution.html?highlight=remappings#import-remapping)) to create a unique set...
I implemented the above remappings scheme in: https://github.com/dapphub/dapptools/pull/719
I think the issue of on disk submodule duplication can be addressed by modifying `dapp update` so that it clones dependencies to a centralized location and then uses the `--reference`...
> This seems like it's really a fault of putting the hash of human-readable relative source in the bytecode (instead of flattened absolute source code, or better yet, no hash...
ah, very interesting! If I understand correctly we would still have the same issue if `A` and `B` imported slightly different versions of oz? We can certainly apply the deduplication...
@deluca-mike I have added a deduplication step the to the remappings generation procedure in https://github.com/dapphub/dapptools/pull/719. I think that with this addtion the new package local remapping are semantically exactly the...