export the storage layout

[charles-cooper]

What I did

Implement #1950

How I did it

How to verify it

Compile with -f layout

(vyper) $ cat tmp/nonreentrant.vy 
bar: uint256

@external
@nonreentrant("lock")
def foo():
    foo: uint256 = 0
(vyper) $ vyper -f layout tmp/nonreentrant.vy  | jq ''
{
  "nonreentrant.lock": {
    "type": "nonreentrant lock",
    "location": "storage",
    "slot": 0
  },
  "bar": {
    "type": "uint256",
    "location": "storage",
    "slot": 1
  }
}

Description for the changelog

Cute Animal Picture

[codecov-commenter]

Codecov Report

Merging #2433 (6d91720) into master (be9d307) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 6d91720 differs from pull request most recent head 99a8961. Consider uploading reports for the commit 99a8961 to get more accurate results

@@            Coverage Diff             @@
##           master    #2433      +/-   ##
==========================================
+ Coverage   84.62%   84.65%   +0.02%     
==========================================
  Files          94       94              
  Lines        9423     9441      +18     
  Branches     2205     2207       +2     
==========================================
+ Hits         7974     7992      +18     
  Misses        939      939              
  Partials      510      510              

Impacted Files	Coverage Δ
vyper/cli/vyper_compile.py	75.22% <ø> (ø)
vyper/cli/vyper_json.py	85.20% <ø> (ø)
vyper/compiler/__init__.py	85.71% <ø> (ø)
vyper/evm/opcodes.py	100.00% <ø> (ø)
vyper/ovm/transpile_lll.py	63.97% <ø> (ø)
vyper/builtin_functions/functions.py	88.48% <100.00%> (ø)
vyper/compiler/output.py	97.16% <100.00%> (+0.06%)	⬆️
vyper/compiler/phases.py	87.85% <100.00%> (+0.59%)	⬆️
vyper/semantics/validation/data_positions.py	93.10% <100.00%> (+3.10%)	⬆️
vyper/typing.py	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be9d307...99a8961. Read the comment docs.

[fubuloubu]

the other ones are just layout?

[charles-cooper]

Not sure why I did this lol but I used "layout" for the cli flag and "storage_layout" internally

[fubuloubu]

I'd say just do one or the other

[charles-cooper]

@fubuloubu I slightly modified some behavior in f792c45, could you take a look? I don't think it's a breaking change but it may deserve its own PR.

[fubuloubu]

@fubuloubu I slightly modified some behavior in f792c45, could you take a look? I don't think it's a breaking change but it may deserve its own PR.

I wouldn't call it a breaking change persay (more of an optimization), but definitely should be it's own PR

[charles-cooper]

@fubuloubu I opened a followup PR for the optimization #2439