[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 --- Comment #9 from Peter A. Bigot --- I haven't used msp430 for several years. Nobody's followed up after the patch was merged, so I'd be ok with closing it. If there's still a problem it's probably a different bug.

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 Martin Liška changed: What|Removed |Added CC||marxin at gcc dot gnu.org --- Comment #8

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 --- Comment #7 from Nick Clifton --- Author: nickc Date: Wed Dec 24 13:36:29 2014 New Revision: 219058 URL: https://gcc.gnu.org/viewcvs?rev=219058&root=gcc&view=rev Log: PR target/64160 * config/msp430/msp430.md (addsi splitter): Do not

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 --- Comment #6 from Nick Clifton --- Hi Ulrich, > if (reg_overlap_mentioned_p (operands[3], operands[7]) > || reg_overlap_mentioned_p (operands[3], operands[8])) >FAIL; Thanks - that is indeed a better solution to the bug. > B.t.w. i

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 --- Comment #5 from Peter A. Bigot --- Thanks, Ulrich: that's a better explanation of what makes me uncomfortable with that part of the machine description. We have a complex split pattern that's the sole user of an idiosyncratic predicate. It

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 --- Comment #4 from Ulrich Weigand --- The ususal test in such scenarios involves reg_overlap_mentioned_p: /* Nonzero if modifying X will affect IN. [...] */ int reg_overlap_mentioned_p (const_rtx x, const_rtx in) which also handles cases lik

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 --- Comment #3 from Peter A. Bigot --- Comment on attachment 34232 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34232 Proposed patch I don't trust that the term nonsubreg is being used correctly in that predicate since the operand does h

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 Nick Clifton changed: What|Removed |Added CC||nickc at redhat dot com --- Comment #2 fr

[Bug target/64160] msp430 code generation error adding 32-bit integers

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64160 Ulrich Weigand changed: What|Removed |Added CC||uweigand at gcc dot gnu.org --- Comment