Skip to content

Commit

Permalink
@wip fix dumb BB error
Browse files Browse the repository at this point in the history
`removeAllPhis` untracking *all* its phis from the CFG, not the ones you actually removed.
  • Loading branch information
Jakobeha committed Jun 21, 2024
1 parent 240be28 commit 6605dc5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/main/java/org/prlprg/ir/cfg/BB.java
Original file line number Diff line number Diff line change
Expand Up @@ -679,7 +679,7 @@ public void removeAllPhis(Collection<? extends Phi<?>> phis) {
if (this.phis.size() != oldSize - phis.size()) {
throw new IllegalArgumentException("Not all in " + id() + ": " + phis);
}
cfg().untrackAll(this.phis);
cfg().untrackAll(phis);

cfg().record(new CFGEdit.RemovePhis(this, phis), new CFGEdit.InsertPhis(this, phis));
}
Expand Down

0 comments on commit 6605dc5

Please sign in to comment.