Probleme mit -O2

Fritz G. (fritzg) OP 24.11.2005 13:11 #283938

Lesenswert?

•

Bin ich doof, oder der Compiler?

Folgender Code:

  beflen = 0;
  pos = 0;
  waitstart = 1;
  ccount = 0;
  parity = 0;
  beftype = 255;
  // 1 heisst warten auf 0 0 Startfolge, erste 0
  // 2 warten auf zweite 0
  // 3 warten auf Befehlstyp, Länge
  do      //mainloop
    while (!(usart_unread_data ()))
      c = usart_getc ();
      if (waitstart != 0)
        if (waitstart == 1)
          if (c == 0)
            waitstart = 2;
        else
          if (waitstart == 2)
            if (c == 0)
              waitstart = 3;
              ccount = 1;
            else
              waitstart = 1;
          else  // muss wohl waitstart 3 sein
            parity += c;
            ccount++;
            switch (ccount)
            case 2:
              beftype = c;
              break;
            case 3:
              shipadr = c << 8;
              break;
            case 4:
              shipadr += c;
              break;
            case 5:
              if ((beftype == 1) || (beftype == 2))
                beflen = c * 2;
                waitstart = 0;
            data[beflen] = c;
      else
        data[beflen] = c;
        parity += c;
        if (pos < beflen) ;
  while (1);


Mit -O2 optimiert er alles nach
c = usart_getc ();
weg.
Ersetze ich das
    while (!(usart_unread_data ()))
durch ein if ... dann tut es.

Hier das Assemblerlisting:

zentrale.c    ****   beflen = 0;
zentrale.c    ****   pos = 0;
zentrale.c    ****   waitstart = 1;
zentrale.c    ****   ccount = 0;
zentrale.c    ****   parity = 0;
zentrale.c    ****   beftype = 255;
zentrale.c    ****   // 1 heisst warten auf 0 0 Startfolge, erste
zentrale.c    ****   // 2 warten auf zweite 0
zentrale.c    ****   // 3 warten auf Befehlstyp, Länge
zentrale.c    ****   do      //mainloop
zentrale.c    ****   {
zentrale.c    ****     while (!(usart_unread_data ()))
               /*DEBUG: 0x7    2  16 */
012a 0E94 0000     call usart_unread_data
               /*DEBUG: 0x9    2  8 */
               /*DEBUG: 0xa    1  24 */
0130 E1F7          brne .L76
zentrale.c    ****     {
zentrale.c    ****       c = usart_getc ();
               /*DEBUG: 0xb    1  16 */
0132 0E94 0000     call usart_getc
               /*DEBUG: 0xd    2  8 */
0136 F9CF          rjmp .L76
               /* epilogue: frame size=0 */
               /* epilogue: noreturn */
               /* epilogue end (size=0) */
               /* function main size 18 (14) */


Weiss da jemand Rat?

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Fritz G. (fritzg) OP 24.11.2005 13:14 #283939

Lesenswert?

•

▲
▼

Im Asseblerlistung fehlt oben noch die Zeile

547                 .L76:

und im C File oben:
  do      //mainloop
  {

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Unbekannter Gast 24.11.2005 13:56 #283940

Lesenswert?

•

▲
▼

Der Compiler hat immer recht. Offensichtlich ist Dein hier geposteter
Code nicht vollständig.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Jörg W. (dl8dtl) Moderator 24.11.2005 14:05 #283941

Lesenswert?

•

▲
▼

Guck dir vor allem das tatsächliche Assemblerlisting an,
nicht den Disassembler-Krempel.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Fritz G. (fritzg) OP 24.11.2005 14:26 #283942

Lesenswert?

•

▲
▼

@Unbekannter:
Der Code ist vollständig genug, glaub mir!

Habs schon gefunden, ein printf() oder sonstwas am Ende reicht, dann
tuts wieder.

War ein typischer Fall von: Schön was er da macht, aber irgendwie
braucht er die Ergebnisse eh nicht, also schmeiss ich alles weg.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Rufus Τ. F. (rufus) 24.11.2005 14:45 #283943

Lesenswert?

•

▲
▼

Der Code ist mit Sicherheit nicht vollständig genug - wie sind denn
beispielsweise die Variablen deklariert? War da nicht was mit
"volatile" oder so?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Fritz G. (fritzg) OP 24.11.2005 15:33 #283944

Lesenswert?

•

▲
▼

Der Rest vom Code:

  uint8_t beflen, pos, waitstart, c, ccount, parity, beftype;
  uint8_t data[256];
  uint16_t shipadr;


Da gibts nix mit volatile, weil da noch keine Variblen mit Interrupt
verändert werden.

In dem Code tut die Art und Größe der Variblen übrigens nichts zur
Sache.

Ich hab mir lediglich das Assemblerlistung zu früh angesehen.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Probleme mit -O2

Antwort schreiben