Jason Merrill <ja...@redhat.com> writes: >> + decl_specs->locations[cpdw_builtin_type_spec] = location; >> if (!decl_specs->type) >> { >> decl_specs->type = type_spec; >> decl_specs->type_definition_p = false; >> - decl_specs->type_location = location; >> + decl_specs->locations[cpdw_type_spec] = location; > > Why do we need cpdw_builtin_type_spec?
To unify the way we access locations for the specifiers. Otherwise, it looks awkward to have a lone decl_specs->type_location for the type specifier, whereas the other specifiers have their locations in decl_specs->locations. But if you don't like that, I'll just revert it. -- Dodji