Can we move this into the for loop?  Or maybe delete the Attribute and Name cases and just let it fall to the "unsupported" default case.

Also, I wouldn't normally comment on it but while I'm here, this could be a good place to use a range for loop if you feel like it (`for (AST_expr* target : node->targets)`).