Kleines Makro, inline asm so richtig?

> #define setbit(BYTE, BIT)     asm("SBI  (BYTE), (BIT)") // SBI P,b

Unter der Voraussetzung, das dein "P" ein I/O-Register ist, dann ist das 
auch die effektivste Methode, dort ein Bit einzuschalten.
Spricht in meinen Augen für den Compiler.
mfG ingo

so stehts mal im instruction set :))

SBI P,b Set Bit in I/O Register I/O(P,b) ← 1

Micha s. schrieb:
> und ihr wisst ja selber was der compiler daraus macht...

Ja, ein SBI - normalerweise. Hast du die Optimierung ausgeschaltet? Oder 
wofür soll das Makro sonst gut sein?

ne optimierung ist auf default 0s

((BYTE) |=  (1<<(BIT)) daraus macht er mit sicherheit kein sbi...

er setzt ne odermaske über ein geshiftetes bit, also minimum 2 befehle 
macht er daraus!

Also bei mir nicht, da macht der Compiler sehr zuverlässig SBI/CBI, wenn 
das Code einspart und möglich ist.

Z.B.

1

#include <avr/io.h>

2

3

int main(void) {

4

    PORTA |= 1<<4;

5

6

    for(;;);

7

}

Kompiliert mit

1

avr-gcc -mmcu=attiny2313 -Wall -Os -S -c foo.c

Dann sieht das so aus in foo.s:

1

main:

2

/* prologue: function */

3

/* frame size = 0 */

4

        sbi 59-32,4

5

.L2:

6

        rjmp .L2

Anmerkung: in der avr-libc gab es auch mal Makros sbi() und cbi(). Die 
sind aber schon seit langer Zeit deprecated und wurden dann entfernt, 
gerade eben da der Compiler schlau genug ist.