Skip to content

Fix "Unreachable code." warning in inline assembly.#16432

Closed
rodiazet wants to merge 2 commits intodevelopfrom
cfg-inline-assembly
Closed

Fix "Unreachable code." warning in inline assembly.#16432
rodiazet wants to merge 2 commits intodevelopfrom
cfg-inline-assembly

Conversation

@rodiazet
Copy link
Copy Markdown
Contributor

@rodiazet rodiazet commented Feb 2, 2026

This PR implements building CFG for inline assembly to fix problem of wrongly Unreachable code. warning issuing.
Fixes #13670

WiP

@rodiazet rodiazet changed the title Fix "Unreachable code." warning. Fix "Unreachable code." warning in inline assembly. Feb 2, 2026
// Create new functions definitions scope.
m_yulFunctionsDefinitions.push_back({});
// Iterate trough functions definitions and assign them new graph nodes. They gonna be attached when called.
for (auto const& statement : _block.statements)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coding style error

@github-actions github-actions Bot added the stale The issue/PR was marked as stale because it has been open for too long. label Feb 16, 2026
@github-actions github-actions Bot closed this Feb 23, 2026
@cameel cameel reopened this Mar 1, 2026
@cameel cameel removed stale The issue/PR was marked as stale because it has been open for too long. closed-due-inactivity labels Mar 1, 2026
@argotorg argotorg deleted a comment from github-actions Bot Mar 1, 2026
@argotorg argotorg deleted a comment from github-actions Bot Mar 1, 2026
@github-actions
Copy link
Copy Markdown

This pull request is stale because it has been open for 14 days with no activity.
It will be closed in 7 days unless the stale label is removed.

@github-actions github-actions Bot added the stale The issue/PR was marked as stale because it has been open for too long. label Mar 16, 2026
@rodiazet rodiazet removed the stale The issue/PR was marked as stale because it has been open for too long. label Mar 16, 2026
@github-actions
Copy link
Copy Markdown

This pull request is stale because it has been open for 14 days with no activity.
It will be closed in 7 days unless the stale label is removed.

@github-actions github-actions Bot added the stale The issue/PR was marked as stale because it has been open for too long. label Mar 31, 2026
@cameel cameel removed the stale The issue/PR was marked as stale because it has been open for too long. label Apr 3, 2026
@github-actions
Copy link
Copy Markdown

This pull request is stale because it has been open for 14 days with no activity.
It will be closed in 7 days unless the stale label is removed.

@github-actions github-actions Bot added the stale The issue/PR was marked as stale because it has been open for too long. label Apr 18, 2026
@github-actions
Copy link
Copy Markdown

This pull request was closed due to a lack of activity for 7 days after it was stale.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

closed-due-inactivity stale The issue/PR was marked as stale because it has been open for too long.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

False negative Warning: Unreachable code. in inline assembly

3 participants