On Fri, Mar 18, 2011 at 3:39 PM, Richard Henderson r...@redhat.com wrote:
On 03/18/2011 02:51 PM, H.J. Lu wrote:
See analysis in:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47502
You're patching the wrong place. See can_combine_p,
which can test for specific sources, rather than
On Fri, Mar 18, 2011 at 12:51 PM, Eric Botcazou ebotca...@adacore.com wrote:
Is this patch OK for trunk?
This has always worked for the other ports AFAIK so I don't think we should
disable it without evaluating the impact on them. If reload has so many
problems with x32, maybe more
On 03/18/2011 01:56 PM, H.J. Lu wrote:
On Fri, Mar 18, 2011 at 12:51 PM, Eric Botcazou ebotca...@adacore.com wrote:
Is this patch OK for trunk?
This has always worked for the other ports AFAIK so I don't think we should
disable it without evaluating the impact on them. If reload has so many
On 03/18/2011 02:51 PM, H.J. Lu wrote:
See analysis in:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47502
You're patching the wrong place. See can_combine_p,
which can test for specific sources, rather than
cant_combine_insn_p which has no access to sources.
r~
Hi Eric,
Is this patch OK for trunk?
Thanks.
H.J.
---
On Thu, Jan 27, 2011 at 6:14 PM, H.J. Lu hongjiu...@intel.com wrote:
It is a bad idea to combine asm statement. This patch disallows it.
Any comments?
Thanks.
H.J.
---
commit a11b95180e53d3a0bc3eabad5594859cea7c3334
Author: H.J.
On Mar 17, 2011, at 20:35, H.J. Lu wrote:
- substitutions of likely-spilled regs, reload might die.
+ substitutions of likely-spilled regs, reload might die. Never
+ combine asm statement.
This has to be statements, a plural.
-Geert
On Thu, Mar 17, 2011 at 8:28 PM, Geert Bosch bo...@adacore.com wrote:
On Mar 17, 2011, at 20:35, H.J. Lu wrote:
- substitutions of likely-spilled regs, reload might die.
+ substitutions of likely-spilled regs, reload might die. Never
+ combine asm statement.
This has to be