Bitfeld alignment und #pragma pack

[pc (Gast)]

Hallo,

ich habe ein Bitfeld, bei dem der IAR-Compiler wegen "illegal struct 
load - too low alignment" einen Fehler rauswirft.

typedef struct
{
  unsigned  RDY_n    :1;  
  unsigned  WEN    :1;  
  unsigned  BP0    :1;  
  unsigned  BP1    :1;  
  unsigned  Res_4_to_6  :3;
  unsigned  WPEN    :1;  
}
T_STATUS_REG;

Stelle ich ein #pragma pack(1) voran kommt die Warnung nicht mehr. Gilt 
das #pragma pack(1) für den gesamten Code oder nur für dieses Struct?

[Rufus Τ. F. (rufus)]

pc schrieb:
> Gilt das #pragma pack(1) für den gesamten Code oder nur für dieses
> Struct?

Es gilt für alles, was folgt.

Du musst auf Deine Strukturdeklaration ein korrespondierendes #pragma 
pack mit dem für den restlichen Code gewünschten Wert folgen lassen.

[pc (Gast)]

Hab gerade ein bisschen rumprobiert und kam zum selben Ergebnis. Es ist 
so wie du sagst. Danke!


Lösung:

#pragma pack(1)

typedef struct
{
  unsigned  RDY_n    :1;  
  unsigned  WEN    :1;  
  unsigned  BP0    :1;  
  unsigned  BP1    :1;  
  unsigned  Res_4_to_6  :3;
  unsigned  WPEN    :1;  
}
T_STATUS_REG;

#pragma pack()

[Mitlesa (Gast)]

Rufus Τ. F. schrieb:
> Es gilt für alles, was folgt.

Nebenfrage dazu:

Gilt das Pragma nur für das aktuelle Modul das compiliert
wird oder global für alle folgenden Übersetzungen?

(überleg, überleg ...: eigentlich wird der Compiler nichts
wissen mit welchen Parametern er zuvor beauftragt wurde zu
compilieren, also wird er beim nächsten Modul die Pragma-
Veränderung vom vorherigen Lauf nicht mehr kennen, richtig?)

[Kaj G. (Firma: RUB) (bloody)]

Mitlesa schrieb:
> Gilt das Pragma nur für das aktuelle Modul das compiliert
> wird oder global für alle folgenden Übersetzungen?
Ohne es genau zu wissen, denke ich, dass es sich auf eine 'translation 
unit' beschraenkt.

[weiss was (Gast)]

pc schrieb:
> unsigned  RDY_n    :1;

Das ist verbesserungsfähig:
unsigned uint8_t RDY_n : 1;

Bei nur unsigned wird pauschal ein unsigned int mit 16 Bit verwendet.

[HildeK (Gast)]

weiss was schrieb:
> Das ist verbesserungsfähig:
> unsigned uint8_t RDY_n : 1;

Das auch!
Wenn schon, dann
uint8_t RDY_n : 1;