public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [avr port] CC0 tracking bugfix
@ 2002-08-13  7:00 Denis Chertykov
  2002-08-13 13:33 ` [avr-gcc-list] " Marek Michalkiewicz
  0 siblings, 1 reply; 3+ messages in thread
From: Denis Chertykov @ 2002-08-13  7:00 UTC (permalink / raw)
  To: gcc-patches; +Cc: Zoltan Kocsi, avr-gcc-list

2002-08-13  Denis Chertykov  <denisc@overta.ru>

	* config/avr/avr.md: Call CC_STATUS_INIT in all peepnoles
        which can change CC0.

Index: avr.md
===================================================================
RCS file: /cvs/gcc/egcs/gcc/config/avr/avr.md,v
retrieving revision 1.34
diff -c -3 -p -r1.34 avr.md
*** avr.md	10 Jul 2002 21:08:41 -0000	1.34
--- avr.md	13 Aug 2002 13:55:08 -0000
***************
*** 1,7 ****
  ;; -*- Mode: Scheme -*-
  ;;   Machine description for GNU compiler,
  ;;   for ATMEL AVR micro controllers.
! ;;   Copyright (C) 1998, 1999, 2000, 2001 Free Software Foundation, Inc.
  ;;   Contributed by Denis Chertykov (denisc@overta.ru)
  
  ;; This file is part of GNU CC.
--- 1,7 ----
  ;; -*- Mode: Scheme -*-
  ;;   Machine description for GNU compiler,
  ;;   for ATMEL AVR micro controllers.
! ;;   Copyright (C) 1998, 1999, 2000, 2001, 2002 Free Software Foundation, Inc.
  ;;   Contributed by Denis Chertykov (denisc@overta.ru)
  
  ;; This file is part of GNU CC.
***************
*** 2403,2408 ****
--- 2403,2409 ----
      && test_hard_reg_class (LD_REGS, operands[1]))"
    "*
  {
+   CC_STATUS_INIT;
    if (test_hard_reg_class (ADDW_REGS, operands[0]))
      output_asm_insn (AS2 (sbiw,%0,1) CR_TAB
  		     AS2 (sbc,%C0,__zero_reg__) CR_TAB
***************
*** 2441,2446 ****
--- 2442,2448 ----
      && test_hard_reg_class (LD_REGS, operands[1]))"
    "*
  {
+   CC_STATUS_INIT;
    if (test_hard_reg_class (ADDW_REGS, operands[0]))
      output_asm_insn (AS2 (sbiw,%0,1), operands);
    else
***************
*** 2472,2477 ****
--- 2474,2482 ----
    "test_hard_reg_class (LD_REGS, operands[0])"
    "*
  {
+   CC_STATUS_INIT;
+   cc_status.value1 = operands[0];
+   cc_status.flags |= CC_OVERFLOW_UNUSABLE;
    output_asm_insn (AS2 (subi,%A0,1), operands);
    switch (avr_jump_mode (operands[1],insn))
    {

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [avr-gcc-list] [avr port] CC0 tracking bugfix
  2002-08-13  7:00 [avr port] CC0 tracking bugfix Denis Chertykov
@ 2002-08-13 13:33 ` Marek Michalkiewicz
  2002-08-13 21:44   ` Denis Chertykov
  0 siblings, 1 reply; 3+ messages in thread
From: Marek Michalkiewicz @ 2002-08-13 13:33 UTC (permalink / raw)
  To: Denis Chertykov; +Cc: gcc-patches, Zoltan Kocsi, avr-gcc-list

Hi,

> 2002-08-13  Denis Chertykov  <denisc@overta.ru>
> 
> 	* config/avr/avr.md: Call CC_STATUS_INIT in all peepnoles
>         which can change CC0.

Thanks.  What do you think - should this be fixed in the stable branch
too?  ("Changes require approval by release manager" - does this apply
to the AVR port too?)

Marek

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [avr-gcc-list] [avr port] CC0 tracking bugfix
  2002-08-13 13:33 ` [avr-gcc-list] " Marek Michalkiewicz
@ 2002-08-13 21:44   ` Denis Chertykov
  0 siblings, 0 replies; 3+ messages in thread
From: Denis Chertykov @ 2002-08-13 21:44 UTC (permalink / raw)
  To: Marek Michalkiewicz
  Cc: Denis Chertykov, gcc-patches, Zoltan Kocsi, avr-gcc-list

Marek Michalkiewicz <marekm@amelek.gda.pl> writes:

> Hi,
> 
> > 2002-08-13  Denis Chertykov  <denisc@overta.ru>
> > 
> > 	* config/avr/avr.md: Call CC_STATUS_INIT in all peepnoles
> >         which can change CC0.
> 
> Thanks.  What do you think - should this be fixed in the stable branch
> too? 

Yes.

> ("Changes require approval by release manager" - does this apply to
> the AVR port too?)

IMHO no, but better to ask release manager.

Denis.

PS: As I understand you want to fix this bug in stable branch ?
I'm very busy now.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2002-08-14  4:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-08-13  7:00 [avr port] CC0 tracking bugfix Denis Chertykov
2002-08-13 13:33 ` [avr-gcc-list] " Marek Michalkiewicz
2002-08-13 21:44   ` Denis Chertykov

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).