On Mon, Jul 22, 2013 at 10:09:00AM -0400, Jason Merrill wrote: > On 07/19/2013 02:45 PM, Marek Polacek wrote: > > /* This type represents an entry in the hash table. */ > > struct ubsan_typedesc > > { > >+ /* This represents the type of a variable. */ > > tree type; > >+ > >+ /* This is the VAR_DECL of the type. */ > > tree decl; > > }; > > What I was looking for was something along the lines of "this hash > table maps from a TYPE to a ubsan type descriptor VAR_DECL for that > type".
Ah. Hopefully it's ok this time around; I already commited previous patch, thus this patchlet. 2013-07-22 Marek Polacek <pola...@redhat.com> * ubsan.c (struct ubsan_typedesc): Improve comment. --- gcc/ubsan.c.mp 2013-07-22 16:14:16.266199381 +0200 +++ gcc/ubsan.c 2013-07-22 16:19:48.335517383 +0200 @@ -31,7 +31,8 @@ along with GCC; see the file COPYING3. #include "ubsan.h" #include "c-family/c-common.h" -/* This type represents an entry in the hash table. */ +/* This type represents an entry in the hash table; this hash table + maps from a TYPE to a ubsan type descriptor VAR_DECL for that type. */ struct ubsan_typedesc { /* This represents the type of a variable. */ Marek