On 10/02/14 07:49, Ilya Enkovich wrote:
On 02 Oct 15:34, Marek Polacek wrote:
On Thu, Oct 02, 2014 at 05:29:15PM +0400, Ilya Enkovich wrote:
Hi,
Seems some new warnings appeared lately and now I see a bootstrap failure for
mpx branch due to missing IPA_REF_CHKP case in switch statement in
process_references function. This function handles varpool_nodes which cannot
have references of this kind. This patch fixes a warning.
Yeah, -Wswitch now works even with enum bit-fields, and use in ipa_ref
is an enum bit-fields.
2014-10-01 Ilya Enkovich <ilya.enkov...@intel.com>
* ipa.c (process_references): Support IPA_REF_CHKP.
Probably s/Support/Handle/.
diff --git a/gcc/ipa.c b/gcc/ipa.c
index 39ee815..78f251a 100644
--- a/gcc/ipa.c
+++ b/gcc/ipa.c
@@ -637,6 +637,8 @@ process_references (varpool_node *vnode,
process_references (dyn_cast<varpool_node *> (ref->referring), written,
address_taken, read, explicit_refs);
break;
+ case IPA_REF_CHKP:
+ gcc_unreachable ();
Looks ok.
Marek
Thanks for looking inot it. Here is a version with modified ChangeLog.
Ilya
--
2014-10-02 Ilya Enkovich <ilya.enkov...@intel.com>
* ipa.c (process_references): Handle IPA_REF_CHKP.
OK.
Jeff