Hi,

this patch makes an assert in gimple_phi_set_arg more strict.

The current assert allows 'index == phi->nargs'. While it's true that there are access functions that currently use the nargs..(capacity-1) inclusive range, this is not one of them (and if it was, the condition should be 'index < phi->capacity').

Bootstrapped and reg-tested on x86_64.

Committed as obvious.

Thanks,
- Tom
Fix assert in gimple_phi_set_arg

2017-07-19  Tom de Vries  <t...@codesourcery.com>

	* gimple.h (gimple_phi_set_arg): Make assert more strict.

---
 gcc/gimple.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/gimple.h b/gcc/gimple.h
index e19cc1c..aba7167 100644
--- a/gcc/gimple.h
+++ b/gcc/gimple.h
@@ -4351,7 +4351,7 @@ gimple_phi_arg (gimple *gs, unsigned index)
 static inline void
 gimple_phi_set_arg (gphi *phi, unsigned index, struct phi_arg_d * phiarg)
 {
-  gcc_gimple_checking_assert (index <= phi->nargs);
+  gcc_gimple_checking_assert (index < phi->nargs);
   phi->args[index] = *phiarg;
 }
 

Reply via email to