diff options
| author | Tom de Vries <tom@codesourcery.com> | 2017-09-16 13:11:43 +0000 |
|---|---|---|
| committer | Tom de Vries <vries@gcc.gnu.org> | 2017-09-16 13:11:43 +0000 |
| commit | ebc6a85e3e8dc9a3e02301a6f9e2216781af565a (patch) | |
| tree | 888bb213198ee08c5156289bb7ed91cf22bbfbcc /gcc/c | |
| parent | a1b99efd4b868cfb5583454cd3d70d54fdafbc33 (diff) | |
| download | gcc-ebc6a85e3e8dc9a3e02301a6f9e2216781af565a.zip gcc-ebc6a85e3e8dc9a3e02301a6f9e2216781af565a.tar.gz gcc-ebc6a85e3e8dc9a3e02301a6f9e2216781af565a.tar.bz2 | |
Fix condition folding in c_parser_omp_for_loop
2017-09-16 Tom de Vries <tom@codesourcery.com>
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
Diffstat (limited to 'gcc/c')
| -rw-r--r-- | gcc/c/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/c/c-parser.c | 9 |
2 files changed, 14 insertions, 1 deletions
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 <tom@codesourcery.com> + + PR c/81875 + * c-parser.c (c_parser_omp_for_loop): Fold only operands of cond, not + cond itself. + 2017-09-15 Joseph Myers <joseph@codesourcery.com> 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: |