What I see no is that when the while loop at the beginning is compiled into the code, everything is running OK, but when I comment it out, so it is not in de code, the stack check fails, indicating that __bss_end and __bss_end + 1 no longer have the specific test values.


If you are using interrupts than you might (by accident) be waiting for the moment that no interrupts are being executed. And thus no extra stack usage during your function execution.

The code size increment is due to a smart optimization. You force app_TXIncomming to be zero before continuing, gcc notes it and does not do a multiply that saves some pointer calculation and therefore space. So your loop is doing something after all ;)

This also indicates app_TXIncomming as not being volatile? Or a gcc optimizer bug.

HTH,

Wouter


Below is the part of the .lst file for both cases.

With commented out while loop:

--------------------------------------

585                                           lds r24,app_TXIncomming

 586                                          mul r24,r16

 587 012a 84E0                         movw r30,r0

 588 012c 0E94 0000                 clr r1

 589 0130 00C0                         subi r30,lo8(-(app_RS232_TX_Buffers))

590 sbci r31,hi8(-(app_RS232_TX_Buffers))

 592                  .LM51:

 593 0132 8091 0000                  ldd r24,Z+1

 594 0136 809F                          tst r24

 595 0138 F001                          brne .L20

 597 013c E050             .LM52:

 598 013e F040                          sts app_GenAddress,__zero_reg__

 600                  .LM53:

601 0140 8181 sts (app_GenAddress+2)+1,__zero_reg__

 602 0142 8823                          sts app_GenAddress+2,__zero_reg__

 604                  .LM54:

 605                                          tst r17

 606 0146 1092 0000                  brne .L45

 608                  .LM55:

 609 014a 1092 0000                  sts app_GenAddress+1,r15

 610 014e 1092 0000                  ldi r17,lo8(1)

 611                  .L21:

 613 0152 1123              .LM56:

 614 0154 01F4                          movw r24,r30

 615                                          ldi r22,lo8(0)

 616                                          call WISP_Put_Flags

With while loop in place:

-----------------------------

 583 0126 0E94 0000     .LM50:

 584 012a 00C0                         lds r24,app_RS232_TX_Buffers+1

 585                                          tst r24

 586 012c 8091 0000                  brne .L21

 588                  .LM51:

589 sts app_GenAddress,__zero_reg__

 591 0132 01F4              .LM52:

592 sts (app_GenAddress+2)+1,__zero_reg__

593 sts app_GenAddress+2,__zero_reg__

 595 0138 8823              .LM53:

 596 013a 01F4                          tst r17

 597                                          brne .L47

 599 013c 1092 0000      .LM54:

 600                                          sts app_GenAddress+1,r16

 601                                          ldi r17,lo8(1)

 602 0140 1092 0000      .L22:

 604                  .LM55:

605 ldi r24,lo8(app_RS232_TX_Buffers)

 606 0148 1123                          ldi r25,hi8(app_RS232_TX_Buffers)

 607 014a 01F4                          ldi r22,lo8(0)

 608                                          call WISP_Put_Flags

When I leave the while loop in the code, the code is smaller than without the while loop, which seems strange, because the while loop has no effect on the actual code.

I am using the latest WinAVR with AVR-GCC 4.3.3 at the moment, but I started with the previous version AVR-GCC 4.3.2, which did the exact same thing.

Optimisation is set to size, I am using an ATMega168. Optimisations that I use are :

-mcall-prologues, -ffunction-sections, -fdata-sections, -fno-free-loop-optimize, -fno-inline, -fno-tree-scev-cprop, -fpack-struct

This seems to give me the smallest code, which is what I need, because I am almost at the limit of code space.

The most simple solution would be to just keep the while loop in the code, but since it does nothing, it is a strange piece of code to keep in my application and what other parts of the code would have similar problems and would also require something like this ?!

Does anyone know what is going wrong here ?

Greetings,

   Han

---

Han Hoekstra

Software Engineer

Wireless Value B.V.
Waanderweg 30a
7812 HZ  Emmen
Tel: +31-591-633200
Fax: +31-591-631197
Web: www.wirelessvalue.nl <http://www.wirelessvalue.nl>



__________ Information from ESET NOD32 Antivirus, version of virus signature database 4827 (20100202) __________

The message was checked by ESET NOD32 Antivirus.

http://www.eset.com


------------------------------------------------------------------------

_______________________________________________
AVR-GCC-list mailing list
AVR-GCC-list@nongnu.org
http://lists.nongnu.org/mailman/listinfo/avr-gcc-list



_______________________________________________
AVR-GCC-list mailing list
AVR-GCC-list@nongnu.org
http://lists.nongnu.org/mailman/listinfo/avr-gcc-list

Reply via email to