On 05/10/2017 05:05 AM, Richard Biener wrote:
On Tue, May 9, 2017 at 9:00 PM, Nathan Sidwell <nat...@acm.org> wrote:
+ +#define TDF_ADDRESS (1 << 3) /* dump node addresses */ this leaves 1 << 2 unused.
Yes, that was intentional (though I suspect my note about it was hidden). As you say, I expect further cleanup and didn't want gratuitous churn. I'll add a comment about bit 2 being free.
Otherwise looks like a great cleanup. You might want to coordinate with Martin a bit here. It also looks like with this we can start re-using bits when they are restricted to one TDF_KIND.
Indeed, we coordinated a bit yesterday. Thanks for review! nathan -- Nathan Sidwell