On Tue, Nov 28, 2023 at 02:07:59PM -0500, Taylor Blau wrote: > +static void clear_bb_commit(struct bb_commit *commit) > +{ > + free(commit->reverse_edges); > + bitmap_free(commit->commit_mask); > + bitmap_free(commit->bitmap); > +} I think these bitmaps may sometimes be NULL. But double-checking bitmap_free(), it sensibly is noop when passed NULL. So this look good to me. -Peff