[Bug tree-optimization/107839] spurious "may be used uninitialized" warning while all uses are under "if (c)"
rguenth at gcc dot gnu.org
gcc-bugzilla@gcc.gnu.org
Thu Nov 24 10:12:30 GMT 2022
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107839
--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
We see
<bb 2> [local count: 3508266]:
if (c_4(D) != 0)
goto <bb 3>; [33.00%]
else
goto <bb 10>; [67.00%]
<bb 10> [local count: 2350538]:
goto <bb 4>; [100.00%]
<bb 4> [local count: 3508266]:
# v_10 = PHI <v_5(D)(10), v_8(3)>
_3 = (unsigned int) v_10;
_12 = _3 * 2;
_1 = (int) _12;
<bb 5> [local count: 354334800]:
if (c_4(D) != 0)
goto <bb 7>; [66.33%]
else
goto <bb 8>; [33.67%]
it's loop invariant motion that hoists the v + v compute out of the loop
and thus outside of its controlling condition. You can see it's careful
to not introduce undefined overflow that is possibly conditionally
executed only but it fails to consider the case of 'v' being conditionally
uninitialized.
It's very difficult to do the right thing here - it might be tempting to
hoist the compute as
if (c)
tem = v+v;
while (1)
if (c)
f(tem);
but apart from the technical problems in invariant motion this would
cause it does introduce another variable that's only conditionally
initialized and thus might be prone to false positive diagnostics.
Not to mention the hoisted if (c) branch having a cost.
Maybe the simplest thing would be to never hoist v + v, or only
hoist it when the controlling branch is not loop invariant.
The original testcase is probably more "sensible", does it still have
a loop invariant controlling condition and a loop invariant computation
under that control?
More information about the Gcc-bugs
mailing list