feat: TryDatabaseCommit

[prestwich]

Adds a fallible version of DatabaseCommit. Idea came up while working on this PR

Basically, this would allow (e.g.) Arc<ConcurrentState<Db>> to be used concurrently for read-access across any number of EVMs (sharing memory cache resources), and then allow Commit access to the BundleState if and only if you have discarded all-but-one Arc. This would enable easier parallel simulation in rounds much easier

[prestwich]

incidentally, this can't be implemented effectively in trevm, as the blanket impl<Db: DatabaseCommit> TryDatabaseCommit for Db would have a potential conflict with theimpl<Db: DatabaseCommit> TryDatabaseCommit for Arc<Db> due to the fact that revm could impl<Db> DatabaseCommit for Arc<Db> in future versions. This is disallowed by the compiler 😮‍💨

[codspeed-hq]

CodSpeed Performance Report

Merging #2121 will not alter performance

_{Comparing prestwich:prestwich/try-commit (86ee82f) with main (739ffd3)}

Summary

✅ 8 untouched benchmarks