[PATCH] PR preprocessor/7263 - Avoid pedantic warnings on system headers macro tokens
Dodji Seketeli
dodji@redhat.com
Fri May 4 16:32:00 GMT 2012
Jason Merrill <jason@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
More information about the Gcc-patches
mailing list