On May 16, 2013, at 5:26 PM, David Edelsohn <dje....@gmail.com> wrote:
> This patch is creating new segfaults for 32 bit POWER AIX.

Thanks for the heads up.  Fixed in r199030.

2013-05-17  Mike Stump  <mikest...@comcast.net>

        PR rtl-optimization/57304
        * web.c (union_match_dups): Ensure that DF_REF_LOC exists before
        accessing DF_REF_REAL_LOC.

Index: web.c
===================================================================
--- web.c       (revision 199016)
+++ web.c       (working copy)
@@ -133,9 +133,10 @@ union_match_dups (rtx insn, struct web_e
       entry = type == OP_IN ? use_entry : def_entry;
       for (; *ref; ref++)
        {
-         if (DF_REF_LOC (*ref) == recog_data.operand_loc[op])
+         rtx *l = DF_REF_LOC (*ref);
+         if (l == recog_data.operand_loc[op])
            break;
-         if (DF_REF_REAL_LOC (*ref) == recog_data.operand_loc[op])
+         if (l && DF_REF_REAL_LOC (*ref) == recog_data.operand_loc[op])
            break;
        }
 
@@ -143,9 +144,10 @@ union_match_dups (rtx insn, struct web_e
        {
          for (ref = use_link, entry = use_entry; *ref; ref++)
            {
-             if (DF_REF_LOC (*ref) == recog_data.operand_loc[op])
+             rtx *l = DF_REF_LOC (*ref);
+             if (l == recog_data.operand_loc[op])
                break;
-             if (DF_REF_REAL_LOC (*ref) == recog_data.operand_loc[op])
+             if (l && DF_REF_REAL_LOC (*ref) == recog_data.operand_loc[op])
                break;
            }
        }
------------------------------

Reply via email to