Re: RFA: PR target/53120, constraint modifier + on operand tied by matching-constraint, 0.

2012-05-09 Thread DJ Delorie
OK to apply ? Ok. Thanks!

Re: RFA: PR target/53120, constraint modifier + on operand tied by matching-constraint, 0.

2012-05-08 Thread nick clifton
Hi DJ, Make sure a match_dup will still match the generated pattern later, I've had problems with match_dup not matching two rtx that rtx_equals() says are the same but not physically the same. I have tried, but failed, to find a way to trigger the use of the bset_qi pattern. :-( I tried

Re: RFA: PR target/53120, constraint modifier + on operand tied by matching-constraint, 0.

2012-05-08 Thread DJ Delorie
#define q ((char *)0x1234) foo(int x) { *q |= (1 (char)x); } $ m32c-elf-gcc -S -O3 nick.c .global _foo _foo: mov.w r1,a0; 20 movhi_op/3 bset4660[a0] ; 11

Re: RFA: PR target/53120, constraint modifier + on operand tied by matching-constraint, 0.

2012-05-03 Thread DJ Delorie
Make sure a match_dup will still match the generated pattern later, I've had problems with match_dup not matching two rtx that rtx_equals() says are the same but not physically the same.

RFA: PR target/53120, constraint modifier + on operand tied by matching-constraint, 0.

2012-05-02 Thread nick clifton
Hi DJ, As pointed out by Hans-Peter in his patch for PR 53120 for the cris backend, the m32c/bitops.md file contains a pattern that uses both the + modifier and the 0 constraint. The patch below is a straightforward fix for this, replacing the 0 constraint with a match_dup. Tested with