diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 0a79692a53f..73339dc2c2c 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2020-03-11 Richard Biener + + * match.pd ((T *)(ptr - ptr-cst) -> &MEM[ptr + -ptr-cst]): + New pattern. + 2020-03-11 Eric Botcazou PR middle-end/93961 diff --git a/gcc/match.pd b/gcc/match.pd index 19df0c404a4..9cb37740f1e 100644 --- a/gcc/match.pd +++ b/gcc/match.pd @@ -1864,6 +1864,15 @@ DEFINE_INT_AND_FLOAT_ROUND_FN (RINT) (if (ptr_difference_const (@0, @1, &diff)) { build_int_cst_type (type, diff); })))) +/* Canonicalize (T *)(ptr - ptr-cst) to &MEM[ptr + -ptr-cst]. */ +(simplify + (convert (pointer_diff @0 INTEGER_CST@1)) + (if (POINTER_TYPE_P (type)) + { build_fold_addr_expr_with_type + (build2 (MEM_REF, char_type_node, @0, + wide_int_to_tree (ptr_type_node, wi::neg (wi::to_wide (@1)))), + type); })) + /* If arg0 is derived from the address of an object or function, we may be able to fold this expression using the object or function's alignment. */ diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 0e8d77ab9dd..11061adaf18 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2020-03-11 Richard Biener + + * gcc.dg/torture/20200311-1.c: New testcase. + 2020-03-11 Matthew Malcomson * lib/scanasm.exp (parse_function_bodies): Lines starting with '@' also diff --git a/gcc/testsuite/gcc.dg/torture/20200311-1.c b/gcc/testsuite/gcc.dg/torture/20200311-1.c new file mode 100644 index 00000000000..ac82b6b7a0b --- /dev/null +++ b/gcc/testsuite/gcc.dg/torture/20200311-1.c @@ -0,0 +1,26 @@ +/* { dg-do run } */ + +struct list { struct list *n; }; + +struct obj { + int n; + struct list l; +} _o; + +struct list _l = { .n = &_o.l }; + +int main(int argc, char *argv[]) +{ + struct obj *o = &_o; + _o.l.n = &_l; + while (&o->l != &_l) + /* Note the following is invoking undefined behavior but in + this kind of "obvious" cases we don't want to break things + unnecessarily and thus we avoid analyzing o as pointing + to nothing via the undefined pointer subtraction. Instead + we canonicalize the pointer subtraction followed by the + pointer conversion to pointer offsetting. */ + o = ((struct obj *)((const char *)(o->l.n) + - (const char *)&((struct obj *)0)->l)); + return 0; +}