diff options
| author | Richard Sandiford <richard.sandiford@arm.com> | 2016-05-09 15:29:03 +0000 |
|---|---|---|
| committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2016-05-09 15:29:03 +0000 |
| commit | b9a28869b9314fb06386f6fd2c52d42644d6618d (patch) | |
| tree | 2054e2e06c427a113b32bb3fab8c101e2ea3aa4d /gcc/tree-affine.c | |
| parent | 75a6b91a67f16467c8ed4224819c343b9fbdcedc (diff) | |
Missing pointer dereference in tree-affine.c
wide_int_constant_multiple_p used:
if (*mult_set && mult != 0)
return false;
to check whether we had previously seen a nonzero multiple, but "mult" is
a pointer to the previous value rather than the previous value itself.
Noticed by inspection while working on another patch, so I don't have a
testcase. I tried adding an assert for combinations that were wrongly
rejected before but it didn't trigger during a bootstrap and regtest.
Tested on x86_64-linux-gnu.
gcc/
* tree-affine.c (wide_int_constant_multiple_p): Add missing
pointer dereference.
From-SVN: r236040
Diffstat (limited to 'gcc/tree-affine.c')
| -rw-r--r-- | gcc/tree-affine.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gcc/tree-affine.c b/gcc/tree-affine.c index 32f23013374..4884241134a 100644 --- a/gcc/tree-affine.c +++ b/gcc/tree-affine.c @@ -769,7 +769,7 @@ wide_int_constant_multiple_p (const widest_int &val, const widest_int &div, if (val == 0) { - if (*mult_set && mult != 0) + if (*mult_set && *mult != 0) return false; *mult_set = true; *mult = 0; |