Skip to content

fix[venom]: fix function inliner clone function#4696

Merged
charles-cooper merged 2 commits intovyperlang:masterfrom
charles-cooper:fix/inliner-clone
Jun 17, 2025
Merged

fix[venom]: fix function inliner clone function#4696
charles-cooper merged 2 commits intovyperlang:masterfrom
charles-cooper:fix/inliner-clone

Conversation

@charles-cooper
Copy link
Copy Markdown
Member

rewrite the IRBasicBlock.parent pointer correctly. this is a well-formedness fix as it doesn't seem to be an observable bug at the moment.

What I did

How I did it

How to verify it

Commit message

Commit message for the final, squashed PR. (Optional, but reviewers will appreciate it! Please see our commit message style guide for what we would ideally like to see in a commit message.)

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

rewrite the IRBasicBlock.parent pointer correctly. this is a
well-formedness fix as it doesn't seem to be an observable bug at the
moment.
@charles-cooper charles-cooper requested a review from harkal June 12, 2025 16:52
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.94%. Comparing base (6b5a7e1) to head (6610231).
⚠️ Report is 33 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4696   +/-   ##
=======================================
  Coverage   92.94%   92.94%           
=======================================
  Files         131      131           
  Lines       19032    19032           
  Branches     3307     3307           
=======================================
+ Hits        17689    17690    +1     
+ Misses        905      904    -1     
  Partials      438      438           

☔ 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.

@charles-cooper charles-cooper marked this pull request as ready for review June 16, 2025 18:57
@charles-cooper charles-cooper changed the title fix[venom]: fix function inliner clone function fix[venom]: fix function inliner clone function Jun 17, 2025
@charles-cooper charles-cooper merged commit 67ea4ee into vyperlang:master Jun 17, 2025
162 checks passed
@charles-cooper charles-cooper deleted the fix/inliner-clone branch June 17, 2025 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants