Overriding Storage Allocator Does Not Handle Stateful Modules

[cyberthirst]

Version Information

• vyper Version (output of vyper --version): commit 4b4e188

Issue description

OverridingStorageAllocator.set_storage_slots_with_overrides() does not
handle stateful module imports and initialization properly and will not
allocate storage and transient storage slots for variables or reentrant
functions defined in a sub-module. The issue is only caught at the code
generation stage and would most likely result in the compiler panicking.

credits: @trocher

[charles-cooper]

should be fixed in #3789