public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
[not found] <bug-16563-4@http.gcc.gnu.org/bugzilla/>
@ 2014-02-16 13:16 ` jackie.rosen at hushmail dot com
0 siblings, 0 replies; 9+ messages in thread
From: jackie.rosen at hushmail dot com @ 2014-02-16 13:16 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
Jackie Rosen <jackie.rosen at hushmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jackie.rosen at hushmail dot com
--- Comment #8 from Jackie Rosen <jackie.rosen at hushmail dot com> ---
*** Bug 260998 has been marked as a duplicate of this bug. ***
Seen from the domain http://volichat.com
Page where seen: http://volichat.com/adult-chat-rooms
Marked for reference. Resolved as fixed @bugzilla.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
[not found] <bug-16563-8949@http.gcc.gnu.org/bugzilla/>
2007-02-14 16:48 ` eweddington at cso dot atmel dot com
@ 2008-06-04 13:07 ` eric dot weddington at atmel dot com
1 sibling, 0 replies; 9+ messages in thread
From: eric dot weddington at atmel dot com @ 2008-06-04 13:07 UTC (permalink / raw)
To: gcc-bugs
------- Comment #7 from eric dot weddington at atmel dot com 2008-06-04 13:06 -------
Bug closed based on comment #5.
--
eric dot weddington at atmel dot com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
Target Milestone|--- |4.0.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
[not found] <bug-16563-8949@http.gcc.gnu.org/bugzilla/>
@ 2007-02-14 16:48 ` eweddington at cso dot atmel dot com
2008-06-04 13:07 ` eric dot weddington at atmel dot com
1 sibling, 0 replies; 9+ messages in thread
From: eweddington at cso dot atmel dot com @ 2007-02-14 16:48 UTC (permalink / raw)
To: gcc-bugs
------- Comment #6 from eweddington at cso dot atmel dot com 2007-02-14 16:48 -------
Same results with GCC 4.1.1:
00000000 <testfunc>:
0: af 92 push r10
2: bf 92 push r11
4: cf 92 push r12
6: df 92 push r13
8: ef 92 push r14
a: ff 92 push r15
c: 0f 93 push r16
e: 1f 93 push r17
10: f6 2e mov r15, r22
12: e4 2e mov r14, r20
14: a0 2e mov r10, r16
16: b1 2e mov r11, r17
18: c2 2e mov r12, r18
1a: d3 2e mov r13, r19
1c: 00 d0 rcall .+0 ; 0x1e <testfunc+0x1e>
1e: 8a e0 ldi r24, 0x0A ; 10
20: 00 d0 rcall .+0 ; 0x22 <testfunc+0x22>
22: 8f 2d mov r24, r15
24: 00 d0 rcall .+0 ; 0x26 <testfunc+0x26>
26: 8b e0 ldi r24, 0x0B ; 11
28: 90 e0 ldi r25, 0x00 ; 0
2a: 00 d0 rcall .+0 ; 0x2c <testfunc+0x2c>
2c: 8e 2d mov r24, r14
2e: 00 d0 rcall .+0 ; 0x30 <testfunc+0x30>
30: 86 e0 ldi r24, 0x06 ; 6
32: 00 d0 rcall .+0 ; 0x34 <testfunc+0x34>
34: 9d 2d mov r25, r13
36: 8c 2d mov r24, r12
38: 7b 2d mov r23, r11
3a: 6a 2d mov r22, r10
3c: 00 d0 rcall .+0 ; 0x3e <testfunc+0x3e>
3e: 1f 91 pop r17
40: 0f 91 pop r16
42: ff 90 pop r15
44: ef 90 pop r14
46: df 90 pop r13
48: cf 90 pop r12
4a: bf 90 pop r11
4c: af 90 pop r10
4e: 08 95 ret
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
2004-07-15 10:30 [Bug c/16563] New: " baphomed at lycos dot de
` (4 preceding siblings ...)
2004-09-08 13:57 ` em at la dot mine dot nu
@ 2004-10-06 16:48 ` berndtrog at yahoo dot com
5 siblings, 0 replies; 9+ messages in thread
From: berndtrog at yahoo dot com @ 2004-10-06 16:48 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From berndtrog at yahoo dot com 2004-10-06 16:48 -------
It looks like with gcc version 4.0.0 20040917 it should
work (again):
00000000 <testfunc>:
0: af 92 push r10
2: bf 92 push r11
4: cf 92 push r12
6: df 92 push r13
8: ef 92 push r14
a: ff 92 push r15
c: 0f 93 push r16
e: 1f 93 push r17
10: f6 2e mov r15, r22
12: e4 2e mov r14, r20
14: a0 2e mov r10, r16
16: b1 2e mov r11, r17
18: c2 2e mov r12, r18
1a: d3 2e mov r13, r19
1c: 00 d0 rcall .+0 ; 0x1e
1e: 8a e0 ldi r24, 0x0A ; 10
20: 00 d0 rcall .+0 ; 0x22
22: 8f 2d mov r24, r15
24: 00 d0 rcall .+0 ; 0x26
26: 8b e0 ldi r24, 0x0B ; 11
28: 90 e0 ldi r25, 0x00 ; 0
2a: 00 d0 rcall .+0 ; 0x2c
2c: 8e 2d mov r24, r14
2e: 00 d0 rcall .+0 ; 0x30
30: 86 e0 ldi r24, 0x06 ; 6
32: 00 d0 rcall .+0 ; 0x34
34: 9d 2d mov r25, r13
36: 8c 2d mov r24, r12
38: 7b 2d mov r23, r11
3a: 6a 2d mov r22, r10
3c: 00 d0 rcall .+0 ; 0x3e
3e: 1f 91 pop r17
40: 0f 91 pop r16
42: ff 90 pop r15
44: ef 90 pop r14
46: df 90 pop r13
48: cf 90 pop r12
4a: bf 90 pop r11
4c: af 90 pop r10
4e: 08 95 ret
...
--
What |Removed |Added
----------------------------------------------------------------------------
CC| |berndtrog at yahoo dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
2004-07-15 10:30 [Bug c/16563] New: " baphomed at lycos dot de
` (3 preceding siblings ...)
2004-08-31 18:35 ` denisc at overta dot ru
@ 2004-09-08 13:57 ` em at la dot mine dot nu
2004-10-06 16:48 ` berndtrog at yahoo dot com
5 siblings, 0 replies; 9+ messages in thread
From: em at la dot mine dot nu @ 2004-09-08 13:57 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From em at la dot mine dot nu 2004-09-08 13:57 -------
I have found another bug which I think is related to this one:
typedef unsigned char u08;
typedef unsigned short u16;
#define PROGMEM __attribute__((__progmem__))
#define pgm_read_word(addr) \
({ \
u16 __addr16 = (u16)(addr); \
u16 __result; \
__asm__ \
( \
"lpm %A0, Z+" "\n\t" \
"lpm %B0, Z" "\n\t" \
: "=r" (__result), "=z" (__addr16) \
: "1" (__addr16) \
); \
__result; \
})
PROGMEM u16 crctbl[1] = { 0x0000 };
u08 crc[2];
void
crc16( u08 * bufptr, u16 len )
{
*(u16*)crc = 0;
for ( ; len--; bufptr++ )
// *(u16*)crc = pgm_read_word( &crctbl[crc[1] ^ *bufptr] ) ^ ( crc[0] << 8 );
*(u16*)crc = ( crc[0] << 8 ) ^ pgm_read_word( &crctbl[crc[1] ^ *bufptr] );
}
int
main( void )
{
return 0;
}
Compiling the above .c generates wrong code for the following line
*(u16*)crc = ( crc[0] << 8 ) ^ pgm_read_word( &crctbl[crc[1] ^ *bufptr] );
9c: 80 91 60 00 lds r24, 0x0060
a0: 99 27 eor r25, r25
a2: 38 2f mov r19, r24
a4: 22 27 eor r18, r18
a6: 20 91 61 00 lds r18, 0x0061 <- r18 is lost here
aa: 9d 91 ld r25, X+
ac: 29 27 eor r18, r25
ae: e2 2f mov r30, r18
b0: ff 27 eor r31, r31
b2: ee 0f add r30, r30
b4: ff 1f adc r31, r31
b6: ec 5a subi r30, 0xAC
b8: ff 4f sbci r31, 0xFF
ba: 85 91 lpm r24, Z+
bc: 94 91 lpm r25, Z
be: 28 27 eor r18, r24 <- uses the wrong value
c0: 39 27 eor r19, r25
c2: 30 93 61 00 sts 0x0061, r19
c6: 20 93 60 00 sts 0x0060, r18
If you change the line and reverse the XOR then the generated code is fine:
*(u16*)crc = pgm_read_word( &crctbl[crc[1] ^ *bufptr] ) ^ ( crc[0] << 8 );
I'm using avr-gcc (GCC) 3.4.1 and the following flags to compile:
CPFLAGS = -g -O3 -funsigned-char -funsigned-bitfields -fpack-struct
-fshort-enums -Wall -W
+strict-prototypes -Wa,-ahlms=$(<:.c=.lst) -mmcu=atmega16
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
2004-07-15 10:30 [Bug c/16563] New: " baphomed at lycos dot de
` (2 preceding siblings ...)
2004-07-15 20:41 ` j dot gnu at uriah dot heep dot sax dot de
@ 2004-08-31 18:35 ` denisc at overta dot ru
2004-09-08 13:57 ` em at la dot mine dot nu
2004-10-06 16:48 ` berndtrog at yahoo dot com
5 siblings, 0 replies; 9+ messages in thread
From: denisc at overta dot ru @ 2004-08-31 18:35 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From denisc at overta dot ru 2004-08-31 18:35 -------
I have analyzed this bug and founded that it's not a target bug.
It's a bug in regrename.c
regrename.c have a wrong tracking of call-clobbered registers.
avr have defined r2, ... r16,r17 as call-saved and r18,r19,... as call-clobbered.
Register allocator (greg) produce right code and copy 'd' to r10,r11,r12,r13.
Register renaming (rnreg) produce a wrong code. Rnreg remove copy insn because
it have a wrong tracking of register r16 in SImode through call_insn's.
regrename.c use the `kill_value_regno' to remove information about all
call-clobbered registers from `value_data' structure, but `kill_value_regno'
can't remove all information
about r16 in SImode (r16,r17,r18,r19) because call-clobbered registers only r18,
r19, ...
In example:
We have a link from r10 to r16 in SImode (for avr port SImoode use r16,r17,r18,r19)
call to `kill_value_regno (18, vd)' do nothing because r18 havn't any link. (r16
have. Not r18 nor r17,r19). I think that it's wrong.
I don't know a right place for fixing code.
`fix_value' probably have a code which can be used as a part for fix.
--
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Ever Confirmed| |1
Last reconfirmed|0000-00-00 00:00:00 |2004-08-31 18:35:16
date| |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
2004-07-15 10:30 [Bug c/16563] New: " baphomed at lycos dot de
2004-07-15 11:57 ` [Bug target/16563] " bangerth at dealii dot org
2004-07-15 20:23 ` ericw at evcohs dot com
@ 2004-07-15 20:41 ` j dot gnu at uriah dot heep dot sax dot de
2004-08-31 18:35 ` denisc at overta dot ru
` (2 subsequent siblings)
5 siblings, 0 replies; 9+ messages in thread
From: j dot gnu at uriah dot heep dot sax dot de @ 2004-07-15 20:41 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From j dot gnu at uriah dot heep dot sax dot de 2004-07-15 20:41 -------
I can confirm the bug, as well as the regression against GCC 3.3.x.
In GCC 3.3.x, parameter `d' (in Martin's test case) has been internally
copied over to r14 through r17, and that value was used then. In GCC 3.4.x,
`d' is no longer saved, so wrong values will be passed on once a called
function has modified r18 and/or r19 (which it is free to do according to the
ABI specification).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
2004-07-15 10:30 [Bug c/16563] New: " baphomed at lycos dot de
2004-07-15 11:57 ` [Bug target/16563] " bangerth at dealii dot org
@ 2004-07-15 20:23 ` ericw at evcohs dot com
2004-07-15 20:41 ` j dot gnu at uriah dot heep dot sax dot de
` (3 subsequent siblings)
5 siblings, 0 replies; 9+ messages in thread
From: ericw at evcohs dot com @ 2004-07-15 20:23 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From ericw at evcohs dot com 2004-07-15 20:23 -------
There has been traffic on the avr-gcc-list mailing list that suggests that this
is a regression against the 3.4 series:
<http://www.avr1.org/pipermail/avr-gcc-list/2004-July/007134.html>
<http://www.avr1.org/pipermail/avr-gcc-list/2004-July/007141.html>
Please note that Joerg Wunsch who did the testing in the first message above is
the FreeBSD maintainer of the AVR toolset.
The ABI for the AVR toolset is described here:
<http://www.nongnu.org/avr-libc/user-manual/FAQ.html#faq_reg_usage>
--
What |Removed |Added
----------------------------------------------------------------------------
CC| |ericw at evcohs dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19
2004-07-15 10:30 [Bug c/16563] New: " baphomed at lycos dot de
@ 2004-07-15 11:57 ` bangerth at dealii dot org
2004-07-15 20:23 ` ericw at evcohs dot com
` (4 subsequent siblings)
5 siblings, 0 replies; 9+ messages in thread
From: bangerth at dealii dot org @ 2004-07-15 11:57 UTC (permalink / raw)
To: gcc-bugs
--
What |Removed |Added
----------------------------------------------------------------------------
Component|c |target
Summary|The compiler doesnt the |The compiler doesnt the
|necessary push/restore of |necessary push/restore of
|r18/r19 |r18/r19
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16563
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2014-02-16 13:16 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <bug-16563-4@http.gcc.gnu.org/bugzilla/>
2014-02-16 13:16 ` [Bug target/16563] The compiler doesnt the necessary push/restore of r18/r19 jackie.rosen at hushmail dot com
[not found] <bug-16563-8949@http.gcc.gnu.org/bugzilla/>
2007-02-14 16:48 ` eweddington at cso dot atmel dot com
2008-06-04 13:07 ` eric dot weddington at atmel dot com
2004-07-15 10:30 [Bug c/16563] New: " baphomed at lycos dot de
2004-07-15 11:57 ` [Bug target/16563] " bangerth at dealii dot org
2004-07-15 20:23 ` ericw at evcohs dot com
2004-07-15 20:41 ` j dot gnu at uriah dot heep dot sax dot de
2004-08-31 18:35 ` denisc at overta dot ru
2004-09-08 13:57 ` em at la dot mine dot nu
2004-10-06 16:48 ` berndtrog at yahoo dot com
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).