Hi, in PR 61340 it has been reported that clang warns about unhandeld enum values in a switch (gcc does not, I guess I'll open a new PR for that).
Fixed thusly by adding a default label with a gcc_unreachable() in both. The potentially unhandled enum value in both cases is IPA_REF_ALIAS which we cannot encounter because 1) in ipa-pure-const.c we are not processing aliases and 2) in ipa-reference.c we analyze references of a function to a variable and that cannot be an alias. OK for trunk? Thanks, Martin 2014-06-03 Martin Jambor <mjam...@suse.cz> PR ipa/61340 * ipa-pure-const.c (propagate_pure_const): Add unreachable default handler for switch on an ipa_ref_use enum. * ipa-reference.c (analyze_function): Likewise. diff --git a/gcc/ipa-pure-const.c b/gcc/ipa-pure-const.c index 0ebfe5d..b9a3d3e 100644 --- a/gcc/ipa-pure-const.c +++ b/gcc/ipa-pure-const.c @@ -1286,6 +1286,8 @@ propagate_pure_const (void) break; case IPA_REF_ADDR: break; + default: + gcc_unreachable (); } better_state (&ref_state, &ref_looping, w_l->state_previously_known, diff --git a/gcc/ipa-reference.c b/gcc/ipa-reference.c index ebff60c..bc58cfa 100644 --- a/gcc/ipa-reference.c +++ b/gcc/ipa-reference.c @@ -483,6 +483,8 @@ analyze_function (struct cgraph_node *fn) break; case IPA_REF_ADDR: break; + default: + gcc_unreachable (); } }