From ebc6a85e3e8dc9a3e02301a6f9e2216781af565a Mon Sep 17 00:00:00 2001 From: Tom de Vries Date: Sat, 16 Sep 2017 13:11:43 +0000 Subject: Fix condition folding in c_parser_omp_for_loop 2017-09-16 Tom de Vries PR c/81875 * c-parser.c (c_parser_omp_for_loop): Fold only operands of cond, not cond itself. * testsuite/libgomp.c-c++-common/pr81875.c: New test. From-SVN: r252873 --- gcc/c/ChangeLog | 6 ++++++ gcc/c/c-parser.c | 9 ++++++++- 2 files changed, 14 insertions(+), 1 deletion(-) (limited to 'gcc/c') diff --git a/gcc/c/ChangeLog b/gcc/c/ChangeLog index 742867a..135445e 100644 --- a/gcc/c/ChangeLog +++ b/gcc/c/ChangeLog @@ -1,3 +1,9 @@ +2017-09-16 Tom de Vries + + PR c/81875 + * c-parser.c (c_parser_omp_for_loop): Fold only operands of cond, not + cond itself. + 2017-09-15 Joseph Myers PR c/82071 diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c index f4e1cf6..a36397b 100644 --- a/gcc/c/c-parser.c +++ b/gcc/c/c-parser.c @@ -15224,7 +15224,14 @@ c_parser_omp_for_loop (location_t loc, c_parser *parser, enum tree_code code, cond = cond_expr.value; cond = c_objc_common_truthvalue_conversion (cond_loc, cond); - cond = c_fully_fold (cond, false, NULL); + if (COMPARISON_CLASS_P (cond)) + { + tree op0 = TREE_OPERAND (cond, 0), op1 = TREE_OPERAND (cond, 1); + op0 = c_fully_fold (op0, false, NULL); + op1 = c_fully_fold (op1, false, NULL); + TREE_OPERAND (cond, 0) = op0; + TREE_OPERAND (cond, 1) = op1; + } switch (cond_expr.original_code) { case GT_EXPR: -- cgit v1.1