Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-22 Thread Ulrich Weigand
Georg-Johann Lay wrote: > Ulrich Weigand schrieb: > > Georg-Johann Lay wrote: > > > >>http://gcc.gnu.org/ml/gcc/2011-08/msg00131.html > >> > >>Are you going to install that patch? Or maybe you already installed it? > > > > No, it isn't approved yet (in fact, it isn't even posted for approval). >

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-21 Thread Georg-Johann Lay
Ulrich Weigand schrieb: Georg-Johann Lay wrote: http://gcc.gnu.org/ml/gcc/2011-08/msg00131.html Are you going to install that patch? Or maybe you already installed it? No, it isn't approved yet (in fact, it isn't even posted for approval). Usually, patches that add new target macros, or new

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-18 Thread Ulrich Weigand
Georg-Johann Lay wrote: > http://gcc.gnu.org/ml/gcc/2011-08/msg00131.html > > Are you going to install that patch? Or maybe you already installed it? No, it isn't approved yet (in fact, it isn't even posted for approval). Usually, patches that add new target macros, or new arguments to target mac

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-17 Thread Georg-Johann Lay
http://gcc.gnu.org/ml/gcc/2011-08/msg00131.html Georg-Johann Lay a écrit: Ulrich Weigand wrote: Georg-Johann Lay wrote: Thanks, it works. OK, thanks for testing! [...] Bye, Ulrich Are you going to install that patch? Or maybe you already installed it? Then, I wonder how the following

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-10 Thread Georg-Johann Lay
Ulrich Weigand wrote: > Georg-Johann Lay wrote: > >> Thanks, it works. > > OK, thanks for testing! > >> std Y+2,r31 ; 30 *movphi/3 [length = 7] >> std Y+1,r30 > > I'm actually not seeing those (maybe I'm using a different code > base than you were using ...) > > But I st

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-09 Thread Ulrich Weigand
Georg-Johann Lay wrote: > Thanks, it works. OK, thanks for testing! > std Y+2,r31 ; 30 *movphi/3 [length = 7] > std Y+1,r30 I'm actually not seeing those (maybe I'm using a different code base than you were using ...) But I still see that the frame is created. The pro

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-08 Thread Georg-Johann Lay
Ulrich Weigand wrote: > Georg-Johann Lay wrote: >> Ulrich Weigand wrote: >>> This means that problems like the one you're seeing have been hidden >>> so far. I've started looking into fixing this, but since I don't >>> have a target where this is needed, this effort never really went >>> anywhere.

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-05 Thread DJ Delorie
Nope, I don't use those :-)

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-05 Thread Ulrich Weigand
DJ Delorie wrote: > Was this reproducible for m32c also? I can test it if so... The patch simply passes the destination address space through to MODE_CODE_BASE_REG_CLASS and REGNO_MODE_CODE_OK_FOR_BASE_P, to allow targets to make register allocation decisions based on address space. As long as

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-05 Thread DJ Delorie
Was this reproducible for m32c also? I can test it if so...

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-05 Thread Ulrich Weigand
Michael Matz wrote: > On Fri, 5 Aug 2011, Ulrich Weigand wrote: > > Instead, if you just have a address and you don't know ahead of time > > whether it refers to Flash or RAM space, you ought to hold that number > > in an "int" (or "short" or whatever integer type is most appropriate), > > and then

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-05 Thread Michael Matz
Hi, On Fri, 5 Aug 2011, Ulrich Weigand wrote: > > However, there are situations like the following where you like to take > > the decision at runtime: > > > > char cast_3 (char in_pgm, void * p) > > { > > return in_pgm ? (*((char __pgm *) p)) : (*((char *) p)); > > } > > That's really an a

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-05 Thread Ulrich Weigand
Georg-Johann Lay wrote: > AVR hardware has basically three address spaces: [snip] OK, thanks for the information! > Of course, RAM and Flash are no subsets of each other when regarded as > physical memory, but they are subsets when regarded as numbers. This > lead to my mistake to define RAM and

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-05 Thread Georg-Johann Lay
Ulrich Weigand schrieb: Georg-Johann Lay wrote: Ulrich Weigand wrote: I'd be happy to bring this up to date if you're willing to work with me to get this tested on a target that needs this support ... Just attached a patch to bugzilla because an AVR user wanted to play with the AS support a

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-05 Thread Ulrich Weigand
Georg-Johann Lay wrote: > Ulrich Weigand wrote: > > I'd be happy to bring this up to date if you're willing to work with > > me to get this tested on a target that needs this support ... > > Just attached a patch to bugzilla because an AVR user wanted to play > with the AS support and asked me to

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-04 Thread Georg-Johann Lay
Ulrich Weigand wrote: > Georg-Johann Lay wrote: > >> Trying to make named address space support work for target AVR, >> I am facing the following problem: >> >> For generic AS, there are three valid base pointer registers >> X , Y and Z. >> >> For the new __pgm AS, only Z is available without offs

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-04 Thread Ulrich Weigand
DJ Delorie wrote: > > The only target supporting named address spaces today is spu-elf, > > m32c-elf does too. Huh, I totally missed that, sorry ... Bye, Ulrich -- Dr. Ulrich Weigand GNU Toolchain for Linux on System z and Cell BE ulrich.weig...@de.ibm.com

Re: [named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-04 Thread DJ Delorie
> The only target supporting named address spaces today is spu-elf, m32c-elf does too.

[named address] ice-on-valid: in postreload.c:reload_cse_simplify_operands

2011-08-04 Thread Georg-Johann Lay
Trying to make named address space support work for target AVR, I am facing the following problem: For generic AS, there are three valid base pointer registers X , Y and Z. For the new __pgm AS, only Z is available without offset. The problem is now that addresses.h:base_reg_class() does not pas