Re: [PATCH] Tree structure marking

2017-10-13 Thread Richard Biener
On Fri, 13 Oct 2017, Nathan Sidwell wrote: > In figuring out a problem with CODE_CONTAINS_STRUCT I noticed that: > > 1) the tree_contains_struct array is unsigned char. bool seems a better > choice now we're in C++-land. > > 2) the MARK_TS_FOO macros used the 'do ... while (0)' idiom. But ther

[PATCH] Tree structure marking

2017-10-13 Thread Nathan Sidwell
In figuring out a problem with CODE_CONTAINS_STRUCT I noticed that: 1) the tree_contains_struct array is unsigned char. bool seems a better choice now we're in C++-land. 2) the MARK_TS_FOO macros used the 'do ... while (0)' idiom. But there's no need for such verbosity. These are 'ary[inde