Commit edc7c4ec by Richard Stallman

(parser_build_binary_op): Replace the assignment of CLASS.

For Z<Y<Z warning, test class of CODE; don't use TREE_CODE (result).

From-SVN: r1921
parent 3bf1c6b5
...@@ -2066,7 +2066,7 @@ convert_arguments (typelist, values, name) ...@@ -2066,7 +2066,7 @@ convert_arguments (typelist, values, name)
In addition to constructing the expression, In addition to constructing the expression,
we check for operands that were written with other binary operators we check for operands that were written with other binary operators
in a way that is likely to confuse the user. */ in a way that is likely to confuse the user. */
tree tree
parser_build_binary_op (code, arg1, arg2) parser_build_binary_op (code, arg1, arg2)
enum tree_code code; enum tree_code code;
...@@ -2132,13 +2132,13 @@ parser_build_binary_op (code, arg1, arg2) ...@@ -2132,13 +2132,13 @@ parser_build_binary_op (code, arg1, arg2)
} }
} }
class = TREE_CODE_CLASS (TREE_CODE (result));
/* Similarly, check for cases like 1<=i<=10 that are probably errors. */ /* Similarly, check for cases like 1<=i<=10 that are probably errors. */
if (class == '<' && extra_warnings if (TREE_CODE_CLASS (code) == '<' && extra_warnings
&& (TREE_CODE_CLASS (code1) == '<' || TREE_CODE_CLASS (code2) == '<')) && (TREE_CODE_CLASS (code1) == '<' || TREE_CODE_CLASS (code2) == '<'))
warning ("comparisons like X<=Y<=Z do not have their mathematical meaning"); warning ("comparisons like X<=Y<=Z do not have their mathematical meaning");
class = TREE_CODE_CLASS (TREE_CODE (result));
/* Record the code that was specified in the source, /* Record the code that was specified in the source,
for the sake of warnings about confusing nesting. */ for the sake of warnings about confusing nesting. */
if (class == 'e' || class == '1' if (class == 'e' || class == '1'
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment