Hi,
On Wed, Nov 24 2021, Jan Hubicka via Gcc wrote:
>> ==5404== Conditional jump or move depends on uninitialised value(s)
>> ==5404== at 0x25DAAD7: incorporate_penalties (ipa-cp.c:3282)
>> ==5404== by 0x25DAAD7: good_cloning_opportunity_p(cgraph_node*, sreal,
>> sreal, profile_count, int) (ipa-cp.c:3340)
>
> I looked at this one (since it is in code I am familiar with) and I
> think it may be real bug. There are flags node_is_self_scc which does
> not seem to be initialized in the constructor.
>
> diff --git a/gcc/ipa-prop.h b/gcc/ipa-prop.h
> index 42842d9466a..1d0c115465c 100644
> --- a/gcc/ipa-prop.h
> +++ b/gcc/ipa-prop.h
> @@ -623,8 +623,8 @@ ipa_node_params::ipa_node_params ()
> : descriptors (NULL), lattices (NULL), ipcp_orig_node (NULL),
> known_csts (vNULL), known_contexts (vNULL), analysis_done (0),
> node_enqueued (0), do_clone_for_all_contexts (0), is_all_contexts_clone
> (0),
> - node_dead (0), node_within_scc (0), node_calling_single_call (0),
> - versionable (0)
> + node_dead (0), node_within_scc (0), node_is_self_scc (0),
> + node_calling_single_call (0), versionable (0)
> {
> }
Oops, can you please commit the change to master and all active
branches?
Thanks,
Martin
