Tom> How about putting it into libiberty? Tom> That way other hashtab users, like gdb, can use it.
Andrew> I have no problem with that, but Jakub didn't seem to think it Andrew> belonged there. All I found was this: http://gcc.gnu.org/ml/gcc-patches/2013-09/msg00721.html Quoting from it: "It doesn't belong to hashtab.h, because that is a libiberty API, this style of iterators is GCC specific." I think that's an accurate assessment of the current code, but I don't see why it has to continue to be that way. My argument in favor of moving it to libiberty is that other programs can then use it; and furthermore that since it is tightly tied to the hashtab implementation, it ought to be maintained there in order to preserve the module boundary. So, please reconsider. thanks, Tom
