On 12/12/12, Jakub Jelinek <ja...@redhat.com> wrote:
> On Tue, Dec 11, 2012 at 02:44:41PM -0800, Lawrence Crowl wrote:
> > +/* Hash a tree in a uid_decl_map. */
> > +
> > +inline hashval_t
> > +uid_decl_hasher::hash (const value_type *item)
> > +{
> > + return item->decl_minimal.uid;
>
> Ugh, why aren't you using DECL_UID here? The fact that DECL_UID is
> right now defined to (DECL_MINIMAL_CHECK (NODE)->decl_minimal.uid)
> might change, we really don't want to update thousands of locations
> should we need to change that.
That is what the code looked like in its original location. I have
resisted making changes not essential to the purpose of the patch.
However, if you want me to change it to use DECL_UID, I will.
--
Lawrence Crowl
