Re: [PATCH 1/2] [C++] Remove tree_list from warn_hidden

2014-12-17 Thread Jason Merrill
On 11/27/2014 01:28 AM, tsaund...@mozilla.com wrote: - if (!base_fndecls) + if (!base_fndecls.exists ()) This should use is_empty rather than exists. OK with that change. Jason

[PATCH 1/2] [C++] Remove tree_list from warn_hidden

2014-11-26 Thread tsaunders
From: Trevor Saunders Hi, I'll claim this is kind of a bug fix in so much as excessive gc allocation and use of tree_list is a bug. bootstrapped + regtested x86_64-unknown-linux-gnu, ok? Trev cp/ * class.c (warn_hidden): Use auto_vec instead of tree_list to hold base_fndecls