On Thu, Oct 8, 2015 at 12:59 PM, Richard Henderson <r...@redhat.com> wrote:
> This is the patch that richi includes in the PR. There will need to
> be an additional patch to solve an ICE for the AVR backend, as noted
> in the PR, but this is good enough to solve the bad-code generation
> problem for the i386 backend.
Hi Richard,
For the record, the root cause is in IVO because it fails to preserve
base object. This patch can only paper over the issue for address
spaces where PTR type and sizetype have the same length, otherwise IVO
generates wrong code which can't be walked around by this patch. I
will take PR66768.
Thanks,
bin
>
>
> * tree-ssa-address.c (create_mem_ref_raw): Retain the correct
> type for the address base.
> ---
> gcc/tree-ssa-address.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/tree-ssa-address.c b/gcc/tree-ssa-address.c
> index 042f9c9..bd10ae7 100644
> --- a/gcc/tree-ssa-address.c
> +++ b/gcc/tree-ssa-address.c
> @@ -388,7 +388,7 @@ create_mem_ref_raw (tree type, tree alias_ptr_type,
> struct mem_address *addr,
> }
> else
> {
> - base = build_int_cst (ptr_type_node, 0);
> + base = build_int_cst (build_pointer_type (type), 0);
> index2 = addr->base;
> }
>
> --
> 2.4.3
>
