Re: SET_EXPR_LOCATION usage for unused tree?

2014-03-14 Thread Richard Biener
On Thu, Mar 13, 2014 at 10:44 PM, Thomas Schwinge tho...@codesourcery.com wrote: Hi! In gcc/c/c-parser.c:c_parser_omp_clause_num_threads (as well as other, similar functions), what is the point of setting the boolean tree c's location, given that this tree won't be used in the following?

SET_EXPR_LOCATION usage for unused tree?

2014-03-13 Thread Thomas Schwinge
Hi! In gcc/c/c-parser.c:c_parser_omp_clause_num_threads (as well as other, similar functions), what is the point of setting the boolean tree c's location, given that this tree won't be used in the following? /* Attempt to statically determine when the number isn't positive. */