On September 29, 2016 4:54:36 PM GMT+02:00, Thomas Schwinge 
<tho...@codesourcery.com> wrote:
>Hi!
>
>While working on something else, I found LTO_STREAMER_DEBUG broken.
>Enabling (#define) gcc/lto-stream.h:LTO_STREAMER_DEBUG, some further
>checking is done on the LTO streamer (writer/reader), and some
>additional
>debugging hooks are provided.  In the end, I didn't use this facility
>for
>my debugging, but here is anyway the patch to un-break it.  Not yet
>thoroughly tested, will do that later -- OK for trunk then?

Works for me though I'd expect the checking code to be broken (I foubd it 
hardly useful)

Richard.

>commit b70b2465f9c901ccf783049ab10dc896a5bb0f67
>Author: Thomas Schwinge <tho...@codesourcery.com>
>Date:   Thu Sep 29 14:30:11 2016 +0200
>
>    Fix LTO_STREAMER_DEBUG build
>---
> gcc/lto-streamer.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
>diff --git gcc/lto-streamer.c gcc/lto-streamer.c
>index bfde1fe..a44a916 100644
>--- gcc/lto-streamer.c
>+++ gcc/lto-streamer.c
>@@ -267,23 +267,23 @@ struct tree_hash_entry
> 
> struct tree_entry_hasher : nofree_ptr_hash <tree_hash_entry>
> {
>-  static inline hashval_t hash (const value_type *);
>-  static inline bool equal (const value_type *, const compare_type *);
>+  static inline hashval_t hash (const tree_hash_entry *);
>+  static inline bool equal (const tree_hash_entry *, const
>tree_hash_entry *);
> };
> 
> inline hashval_t
>-tree_entry_hasher::hash (const value_type *e)
>+tree_entry_hasher::hash (const tree_hash_entry *e)
> {
>   return htab_hash_pointer (e->key);
> }
> 
> inline bool
>-tree_entry_hasher::equal (const value_type *e1, const compare_type
>*e2)
>+tree_entry_hasher::equal (const tree_hash_entry *e1, const
>tree_hash_entry *e2)
> {
>   return (e1->key == e2->key);
> }
> 
>-static hash_table<tree_hash_entry> *tree_htab;
>+static hash_table<tree_entry_hasher> *tree_htab;
> #endif
> 
> /* Initialization common to the LTO reader and writer.  */
>@@ -299,7 +299,7 @@ lto_streamer_init (void)
>     streamer_check_handled_ts_structures ();
> 
> #ifdef LTO_STREAMER_DEBUG
>-  tree_htab = new hash_table<tree_hash_entry> (31);
>+  tree_htab = new hash_table<tree_entry_hasher> (31);
> #endif
> }
> 
>
>
>Grüße
> Thomas


Reply via email to