refactor[venom]: use InstUpdater in more passes

[charles-cooper]

What I did

How I did it

How to verify it

Commit message

move `InstUpdater` to its own file and use it in more passes - it's
safer and faster for passes which use the `dfg` to modify it in place
using `InstUpdater`, rather than modifying instructions (without
updating the `dfg`), and then relying on a stale `dfg`.

Description for the changelog

Cute Animal Picture

[codecov]

Codecov Report

Attention: Patch coverage is 97.29730% with 3 lines in your changes missing coverage. Please review.

Project coverage is 92.15%. Comparing base (ded6b2c) to head (a520fa6).
Report is 85 commits behind head on master.

Files with missing lines	Patch %	Lines
vyper/venom/passes/machinery/inst_updater.py	93.47%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4508      +/-   ##
==========================================
- Coverage   92.16%   92.15%   -0.02%     
==========================================
  Files         122      123       +1     
  Lines       17471    17469       -2     
  Branches     2959     2957       -2     
==========================================
- Hits        16103    16099       -4     
- Misses        957      958       +1     
- Partials      411      412       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[HodanPlodky]

Should not this consume the dfg/call constructor?

[charles-cooper]

hm, looks like it's not actually used

[charles-cooper]

9c4b51b