Code Optimierung [ASM/C]

[Sebastian (Gast)]

Hallo Leute

Ich habe das Problem das ich für den unten gelisteten Code nur 48 Byte 
Speicher zu verfügung habe. So wie er jetzt ist benötigt er aber 52 
Byte.
Ich benutze einen ARM Cortex-M1 und den armcc Compiler. Die verschidenen 
Optimierungs Optionen wie -Ospace und -03 liefern alle den selben ASM 
Code.
Hat noch jemand eine Idee wie man den Code noch klein bekommen könnte?

Vielen Dank.

Sebastian

void reset_handler(void)
  {
  
/* bist is not executed if test pin is set HIGH */
  if(RReg32(SYS_CTRL)&SYS_CTRL__testmode) {}
  else {
    /* do start-up DMA/BIST if the according bit is set */
    if (RReg32(SYS_CONFIG)&SYS_CONFIG__bist_en_trim) { 
      do_start_up_dma();
    }
  }
  
  /* wait until shutdown bit gets cleared */
         while (RReg32(SYS_CTRL)&SYS_CTRL__shutdown) { }
     /* do start-up RAM_BIST if the according bit is set */
         if (RReg32(0x0000C7F7)&0x04000000) {
       do_start_up_ram_bist();
         }
  main();
}

; generated by ARM/Thumb C/C++ Compiler, RVCT3.1 [Build 794]
; commandline armcc [-c --asm -o../software/ram_bist/reset_handler.o --cpu=Cortex-M1 -O3 --data_reorder -Oautoinline -Oinline --diag_suppress=2874 -I../software/CMSIS/Core/CM0/ -I../software/include/ -I../software/drivers/ -I../software/ram_bist/ -DMSC_CLOCK -DITERATIONS=5 ../software/drivers/reset_handler.c]
        THUMB
        REQUIRE8
        PRESERVE8

        AREA ||.text||, CODE, READONLY, ALIGN=2

reset_handler PROC
        PUSH     {r4,lr}
        MOVS     r4,#1
        LSLS     r4,r4,#30
        LDR      r0,[r4,#0]
        LSLS     r0,r0,#27
        BMI      |L1.22|
        LDR      r0,[r4,#0x10]
        LSLS     r0,r0,#29
        BPL      |L1.22|
        BL       do_start_up_dma
|L1.22|
        LDR      r0,[r4,#0]
        LSLS     r0,r0,#30
        BMI      |L1.22|
        LDR      r0,|L1.48|
        LDR      r0,[r0,#0x34]
        LSLS     r0,r0,#5
        BPL      |L1.40|
        BL       do_start_up_ram_bist
|L1.40|
        BL       main
        POP      {r4,pc}
        ENDP

        DCW      0x0000
|L1.48|
        DCD      0x0000c7c3

        AREA ||.arm_vfe_header||, DATA, READONLY, NOALLOC, ALIGN=2

        DCD      0x00000000

        EXPORT reset_handler

        IMPORT ||Lib$$Request$$armlib|| [WEAK]
        IMPORT main
        IMPORT do_start_up_ram_bist
        IMPORT do_start_up_dma

        ASSERT {ENDIAN} = "little"
        ASSERT {INTER} = {TRUE}
        ASSERT {ROPI} = {FALSE}
        ASSERT {RWPI} = {FALSE}
        ASSERT {IEEE_FULL} = {FALSE}
        ASSERT {IEEE_PART} = {FALSE}
        ASSERT {IEEE_JAVA} = {FALSE}
        END

[Ulrich (Gast)]

Da es nur um maximal 48 Bytes geht, wäre inline ASM ein Möglichkeit. 
Wenn es damit nicht geht, braucht man halt einen µC mit mehr 48 bytes 
Speicher (bzw. spart an einer anderen Stelle des Programms).

[Peter D. (pdiener)]

Ich unterstelle mal, dass main() nie beendet wird.

Dann wäre
POP      {r4,pc}
unnötig.

Vermutlich ist in Register r4 am Anfang nichts Sinnvolles gespeichert, 
somit muss es auch nicht gerettet werden.
Damit ist
PUSH     {r4,lr}
vermutlich auch unnötig.

Damit man den Teil dazwischen vernünftig versteht, sollte man die 
Sourcezeilen mit in das listing schreiben lassen. Es ist schon möglich, 
dass man da noch was sparen kann.

Grüße,

Peter