On 04/17/14 10:43, Kyrill Tkachov wrote:
Hi all,
While looking at the build logs I noticed a warning while building
tree-ssa-loop-ivcanon.c about a potential use of an uninitialised variable.
This patchlet fixes that warning by initialising it to 0.
Tested arm-none-eabi.
Ok for trunk?
2014-04-17 Kyrylo Tkachov <kyrylo.tkac...@arm.com>
* tree-ssa-loop-ivcanon.c (canonicalize_loop_induction_variables):
Initialise n_unroll to 0.
So did you do a thorough analysis to verify that it was a false positive
or that it was a valid warning?
For the former, it would be helpful if you filed a BZ. We track these
false positives for -Wuninitialized and do try to correct them. If the
latter then is using 0 for the initializer the safe thing to do?
Jeff