In PhaseChaitin::gather_lrg_masks, several variables have the same name in nested scopes, it looks like following code snippet.
{
  A a;
  {
    A a;
  }
}

This is not helpful to code readability, in particular in a long method like `gather_lrg_masks`, better to rename them.