Re: Fix address space computation in expand_debug_expr

2014-06-05 Thread Jeff Law
On 06/05/14 06:19, Senthil Kumar Selvaraj wrote: gcc/ChangeLog 2014-06-05 Senthil Kumar Selvaraj PR target/52472 * cfgexpand.c (expand_debug_expr): Use address space of nested TREE_TYPE for ADDR_EXPR and MEM_REF. gcc/testsuite/ChangeLog 2014-06-05 Senthil Kumar Selv

Re: Fix address space computation in expand_debug_expr

2014-06-05 Thread Senthil Kumar Selvaraj
On Thu, Jun 05, 2014 at 09:46:25AM +0200, Richard Biener wrote: > On Wed, Jun 4, 2014 at 10:06 PM, Senthil Kumar Selvaraj > wrote: > > For the AVR target, assertions in convert_debug_memory_address cause a > > couple of ICEs (see PR 52472). Jakub suggested returning a NULL rtx, > > which works, bu

Re: Fix address space computation in expand_debug_expr

2014-06-05 Thread Richard Biener
On Wed, Jun 4, 2014 at 10:06 PM, Senthil Kumar Selvaraj wrote: > For the AVR target, assertions in convert_debug_memory_address cause a > couple of ICEs (see PR 52472). Jakub suggested returning a NULL rtx, > which works, but on debugging further, I found that expand_debug_expr > appears to incorr

Fix address space computation in expand_debug_expr

2014-06-04 Thread Senthil Kumar Selvaraj
For the AVR target, assertions in convert_debug_memory_address cause a couple of ICEs (see PR 52472). Jakub suggested returning a NULL rtx, which works, but on debugging further, I found that expand_debug_expr appears to incorrectly compute the address space for ADDR_EXPR and MEM_REFs. For ADDR_EX