On 06/02/2014 12:23 PM, Marek Polacek wrote:
I have no special reason for that check, so dropped it too in this patch.
Thanks. I expect that warn_logical_not_parentheses will crash if one of the operands is type-dependent such that TREE_TYPE is NULL_TREE, so you'll want to handle that case, and test it in the testcase, i.e. something like
#ifdef __cplusplus template <class T, class U> bool f(T t, U u) { return (!t == u); } #endif I think !t should have null TREE_TYPE in this case. Jason