avr-gcc: Optimierung: mehrere RCALLs erzeugt

[N. G. (newgeneration)]

Hallo Forum,

Testcode:

extern uint8_t x;
extern void bar(const uint8_t *cmd);

void foo(void) {
  uint8_t cmd[6];
  cmd[0] = x;
  cmd[1] = 0x00;
  cmd[2] = 0x00;
  cmd[3] = 0x00;
  cmd[4] = 0x00;
  cmd[5] = 0xFF;
  bar(cmd);
}

wird mit avr-gcc -s -Os -mmcu=avr6

foo:
  push r28
  push r29
  rcall .            // !
  rcall .            // !
  in r28,__SP_L__
  in r29,__SP_H__
/* prologue: function */
/* frame size = 6 */
/* stack size = 8 */
.L__stack_usage = 8
  lds r24,x
  std Y+1,r24
  std Y+2,__zero_reg__
  std Y+3,__zero_reg__
  std Y+4,__zero_reg__
  std Y+5,__zero_reg__
  ldi r24,lo8(-1)
  std Y+6,r24
  movw r24,r28
  adiw r24,1
  call bar
/* epilogue start */
  adiw r28,6
  in __tmp_reg__,__SREG__
  cli
  out __SP_H__,r29
  out __SREG__,__tmp_reg__
  out __SP_L__,r28
  pop r29
  pop r28
  ret
  .ident  "GCC: (GNU) 5.3.1 20160106"

Wie man sieht werden im Prolog der Funktion 2x RCALL . erzeugt.

Ich kann mir das nicht erklären.

mit freundlichen Grüßen und frohe Weihnachten,
N.G.

[Bilch (Gast)]

Einfach durch den Simulator steppen, dann siehst du schon was passiert.

[Rolf M. (rmagnus)]

N. G. schrieb:
> Wie man sieht werden im Prolog der Funktion 2x RCALL . erzeugt.
>
> Ich kann mir das nicht erklären.

Die zwei RCALLS dienen lediglich dazu, 6 Bytes Platz auf dem Stack für 
dein lokales Array zu schaffen. Das dient der Optimierung der Codegröße, 
da es das mit lediglich zwei Instruktionen schafft. Alle anderen 
Möglichkeiten wären größer.

[N. G. (newgeneration)]

Rolf M. schrieb:
> Die zwei RCALLS dienen lediglich dazu, 6 Bytes Platz auf dem Stack für
> dein lokales Array zu schaffen. Das dient der Optimierung der Codegröße,
> da es das mit lediglich zwei Instruktionen schafft. Alle anderen
> Möglichkeiten wären größer.

Ach so okay, danke!

Diese Möglichkeit kannte ich noch nicht.

Das wäre dann schon geklärt, so einfach kanns manchmal sein.

Danke und frohe Weihnachten
N.G.

[Peter D. (peda)]

Rolf M. schrieb:
> Die zwei RCALLS dienen lediglich dazu, 6 Bytes Platz auf dem Stack für
> dein lokales Array zu schaffen.

Da gilt aber nur für den ATmega2560/1. Auf kleineren AVRs pusht RCALL 
nur 2 Byte.

[N. G. (newgeneration)]

Peter D. schrieb:
> Rolf M. schrieb:
>> Die zwei RCALLS dienen lediglich dazu, 6 Bytes Platz auf dem Stack für
>> dein lokales Array zu schaffen.
>
> Da gilt aber nur für den ATmega2560/1. Auf kleineren AVRs pusht RCALL
> nur 2 Byte.

Ja, das gilt nur für AVRs mit der avr6-Architektur (alle mit 256kB 
Flash, z.B. auch der ATmega256rfr2), da diese einen PC von > 16bit (= 3 
Byte) haben.
Auf einem kleineren AVR mit nur 2 Byte Program Counter sind es auch 
wirklich 3x RCALLs.

[Rolf M. (rmagnus)]

N. G. schrieb:
> Ja, das gilt nur für AVRs mit der avr6-Architektur

Du hattest ja geschickterweise erwähnt, dass sich um so einen handelt:

N. G. schrieb:
> wird mit avr-gcc -s -Os -mmcu=avr6

Ich hatte anfangs auch gedacht, dass da doch eigentlich noch 2 Bytes 
fehlen, und dann hab ich obige Zeile gesehen.