I got tired of writing strcmp (IDENTIFIER_POINTER and decided to wrap
it in an inline function.  I decided to use "id_strcmp" instead of
just overloading strcmp, but I don't feel strongly about that choice.

The second patch changes all existing uses of that pattern to use the
new function.

OK for trunk?