On Wed, Aug 1, 2012 at 9:52 AM, Alan Modra <amo...@gmail.com> wrote: > Hi David, > This is the last of my vendetta against the "o" constraint in the > rs6000 backend. Remaining are a few places in rs6000.md where the > operand predicate is offsettable_mem_operand, and one instance in > spe.md. I believe none of these can run into mode_dependent_address > causing "insn does not statisfy its constraints". Also, with "Y" > replacing "o" everywhere a 64-bit gpr is loaded or stored, we can > remove the +0 wrapper you added for pr13674. > > I also add some testcases with this patch. dfmode_off and tfmode_off > will fail on e500, but the generated code at -O2 is so awful that I'm > inclined to think it should remain that way until someone fixes it! > > Bootstrapped and regression tested powerpc64-linux. OK to apply? > > gcc/ > * config/rs6000/rs6000.c (legitimize_reload_address): Remove code > handling non-aligned ld/std. > * config/rs6000/paired.md (movv2sf_paired): Use 'Y' instead of 'o'. > * config/rs6000/vsx.md (vsx_mov, vsx_movti): Likewise. > * config/rs6000/altivec.md (altivec_mov, altivec_movti): Likewise. > * config/rs6000/dfp.md (movtd_internal): Use 'm' instead of 'o'. > gcc/testsuite/ > * gcc.target/powerpc/dimode_off.c: New. > * gcc.target/powerpc/timode_off.c: New. > * gcc.target/powerpc/dfmode_off.c: New. > * gcc.target/powerpc/tfmode_off.c: New.
This is okay. Thanks, David