On 10/13/14 18:16, Joern Rennecke wrote:
On 13 October 2014 20:43, Jeff Law l...@redhat.com wrote: ...
I think you want smode in the mode_for_size call rather than
mode, right (both instances)?
No, nregs is the number of hard registers of regno in mode. Hence
we must use the size of mode.
On 10/11/14 03:32, Joern Rennecke wrote:
On 10 October 2014 21:13, Jeff Law l...@redhat.com wrote:
...
ISTM it would be better to find the mode of the same class that corresponds
to GET_MODE_SIZE (mode) / nregs. In your case that's obviously QImode :-)
Like this?
Or did you mean to remove
On 13 October 2014 20:43, Jeff Law l...@redhat.com wrote:
...
I think you want smode in the mode_for_size call rather than mode, right
(both instances)?
No, nregs is the number of hard registers of regno in mode. Hence
we must use the
size of mode.
How to choose the mode class is not so
On 10 October 2014 21:13, Jeff Law l...@redhat.com wrote:
...
ISTM it would be better to find the mode of the same class that corresponds
to GET_MODE_SIZE (mode) / nregs. In your case that's obviously QImode :-)
Like this?
Or did you mean to remove the save_mode[regno] use altogether? I can
On 10/07/14 15:56, Joern Rennecke wrote:
On 7 October 2014 18:38, Jeff Law l...@redhat.com wrote:
On 10/06/14 20:57, Joern Rennecke wrote:
On 6 October 2014 19:58, Jeff Law l...@redhat.com wrote:
What makes word_mode special here? ie, why is special casing for
word_mode
the right thing to
On 10/06/14 20:57, Joern Rennecke wrote:
On 6 October 2014 19:58, Jeff Law l...@redhat.com wrote:
What makes word_mode special here? ie, why is special casing for word_mode
the right thing to do?
The patch does not special-case word mode. The if condition tests if
smode would
cover multiple
On 7 October 2014 18:38, Jeff Law l...@redhat.com wrote:
On 10/06/14 20:57, Joern Rennecke wrote:
On 6 October 2014 19:58, Jeff Law l...@redhat.com wrote:
What makes word_mode special here? ie, why is special casing for
word_mode
the right thing to do?
The patch does not special-case
Investigating an ICE while trying to compile libgcc2.c:__udivmoddi4 for
a new avr variant with different register set/allocation order, I found
replace_reg_with_saved_mem falling over its own nonsense. The instruction:
(debug_insn 97 96 98 2 (var_location:SI __x2 (mult:SI (lshiftrt:SI
(reg/v:SI
On 10/06/14 02:51, Joern Rennecke wrote:
Investigating an ICE while trying to compile libgcc2.c:__udivmoddi4 for
a new avr variant with different register set/allocation order, I found
replace_reg_with_saved_mem falling over its own nonsense. The instruction:
(debug_insn 97 96 98 2
On 6 October 2014 19:58, Jeff Law l...@redhat.com wrote:
What makes word_mode special here? ie, why is special casing for word_mode
the right thing to do?
The patch does not special-case word mode. The if condition tests if
smode would
cover multiple hard registers.
If that would be the case,
10 matches
Mail list logo