Skip to content

Avoid moving labels in a block#202

Merged
sipma merged 1 commit intostatic-analysis-engineering:masterfrom
brk:empty-instr-labels
Mar 12, 2025
Merged

Avoid moving labels in a block#202
sipma merged 1 commit intostatic-analysis-engineering:masterfrom
brk:empty-instr-labels

Conversation

@brk
Copy link
Copy Markdown
Contributor

@brk brk commented Mar 11, 2025

When an empty instrs node in the middle of a block had associated labels, we were moving those labels to the start of the block, which is incorrect. The labels should be kept in order.

When an empty instrs node in the middle of a block had associated
labels, we were moving those labels to the start of the block,
which is incorrect. The labels should be kept in order.
Copy link
Copy Markdown
Contributor

@sipma sipma left a comment

Choose a reason for hiding this comment

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

Thank you!

@sipma sipma merged commit 29fcc82 into static-analysis-engineering:master Mar 12, 2025
@brk brk deleted the empty-instr-labels branch March 21, 2025 05:06
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