diff options
author | Vladimir N. Makarov <vmakarov@redhat.com> | 2021-03-05 11:41:25 -0500 |
---|---|---|
committer | Vladimir N. Makarov <vmakarov@redhat.com> | 2021-03-05 11:44:06 -0500 |
commit | 9105757a59b890194ebf5b4fcbacd58db34ef332 (patch) | |
tree | 6078c3050951a6d80ed41effc1954f87182f5926 /gcc | |
parent | b8188b7d7382e4a74af5dd6a125e76e8d43a68a5 (diff) | |
download | gcc-9105757a59b890194ebf5b4fcbacd58db34ef332.zip gcc-9105757a59b890194ebf5b4fcbacd58db34ef332.tar.gz gcc-9105757a59b890194ebf5b4fcbacd58db34ef332.tar.bz2 |
[PR99378] LRA: Skip decomposing address for asm insn operand with unknown constraint.
Function get_constraint_type returns CT__UNKNOWN for empty constraint
and CT_FIXED_FORM for "X". So process_address_1 skipped
decompose_mem_address only for "X" constraint. To do the same for empty
constraint, skip decompose_mem_address for CT__UNKNOWN.
gcc/ChangeLog:
PR target/99378
* lra-constraints.c (process_address_1): Skip decomposing address
for asm insn operand with unknown constraint.
gcc/testsuite/ChangeLog:
PR target/99378
* gcc.target/i386/pr99123-2.c: New.
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/lra-constraints.c | 5 | ||||
-rw-r--r-- | gcc/testsuite/gcc.target/i386/pr99123-2.c | 51 |
2 files changed, 54 insertions, 2 deletions
diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c index 51acf7f..9253690 100644 --- a/gcc/lra-constraints.c +++ b/gcc/lra-constraints.c @@ -3450,8 +3450,9 @@ process_address_1 (int nop, bool check_only_p, i.e. bcst_mem_operand in i386 backend. */ else if (MEM_P (mem) && !(INSN_CODE (curr_insn) < 0 - && get_constraint_type (cn) == CT_FIXED_FORM - && constraint_satisfied_p (op, cn))) + && (cn == CONSTRAINT__UNKNOWN + || (get_constraint_type (cn) == CT_FIXED_FORM + && constraint_satisfied_p (op, cn))))) decompose_mem_address (&ad, mem); else if (GET_CODE (op) == SUBREG && MEM_P (SUBREG_REG (op))) diff --git a/gcc/testsuite/gcc.target/i386/pr99123-2.c b/gcc/testsuite/gcc.target/i386/pr99123-2.c new file mode 100644 index 0000000..def4eae --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr99123-2.c @@ -0,0 +1,51 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -funroll-loops" } */ + +static inline void * +baz (void *s, unsigned long c, unsigned int count) +{ + int d0, d1; + __asm__ __volatile__ ("" + : "=&c" (d0), "=&D" (d1) + :"a" (c), "q" (count), "0" (count / 4), "" ((long) s) /// "1" + :"memory"); + return s; +} + +struct A +{ + unsigned long *a; +}; + +inline static void * +bar (struct A *x, int y) +{ + char *ptr; + + ptr = (void *) x->a[y >> 12]; + ptr += y % (1UL << 12); + return (void *) ptr; +} + +int +foo (struct A *x, unsigned int *y, int z, int u) +{ + int a, b, c, d, e; + + z += *y; + c = z + u; + a = (z >> 12) + 1; + do + { + b = (a << 12); + d = b - z; + e = c - z; + if (e < d) + d = e; + baz (bar (x, z), 0, d); + z = b; + a++; + } + while (z < c); + return 0; +} |