On 11/18/2015 11:20 PM, Senthil Kumar Selvaraj wrote:
On Wed, Nov 18, 2015 at 09:36:21AM +0100, Richard Biener wrote:
Otherwise ok.
See modified patch below. If you think vrp98.c is unnecessary, feel free
to dump it :).
If ok, could you commit it for me please? I don't have commit access.
Regards
Senthil
gcc/ChangeLog
2015-11-19 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com>
* tree.h (desired_pro_or_demotion_p): New function.
* tree-vrp.c (simplify_cond_using_ranges): Call it.
gcc/testsuite/ChangeLog
2015-11-19 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com>
* gcc.dg/tree-ssa/vrp98.c: New testcase.
* gcc.target/avr/uint8-single-reg.c: New testcase.
I went ahead and committed this as-is.
I do think the vrp98 testcase is useful as it verifies that VRP is doing
what we want in a target independent way. It's a good complement to the
AVR specific testcase.
Thanks,
Jeff