optimiert der Compiler so etwas weg?

[Sven (Gast)]

Hallo,

ich möchte mein Programm etwas "allgemein und portabel" gestalten. Daher 
möchte ich, je nachdem welche FCPU später verwendet wird, den 
entsprechenden ADC-Vorteiler setzen.

Beispielsmäßig so:

#ifndef F_CPU
#define F_CPU           16000000UL                   
#warning kein F_CPU definiert
#endif

  if (F_CPU == 8000000UL){
  ADCSRA = (1<<ADPS2) | (1<<ADPS1);     // Frequenzvorteiler auf 64 einstellen --> 125kHz Abtastrate
  }
  
  
  if (F_CPU == 16000000UL){
  ADCSRA = (1<<ADPS2) |(1<<ADPS1) | (1<<ADPS0);     // Frequenzvorteiler 128 auf einstellen --> 125kHz Abtastrate
  }

Optimiert der Compiler so etwas weg? FCPU ist ja schon zur Compilzeit 
fest und verändert sich während der Laufzeit nicht. Oder wird die 
if-Anweisung tatsächlich während der Laufzeit erst behandelt?

Danke.

[(prx) A. K. (prx)]

Wird vom Compiler wegoptimiert.

Wenn keine syntaktischen Probleme dagegen sprechen, dann sollte man an 
Stelle von hässlichen #if/#else Orgien solchen lesbareren C Code 
verwenden.

[Wolfgang (Gast)]

Sven schrieb:
> Optimiert der Compiler so etwas weg?

Das hängt davon ab, was du als Optimierungsstufe gewählt hast. In 
solchen Fällen kann man einfach im List-File (.LSS) nachsehen, was der 
Compiler daraus generiert hat.

[Troll (Gast)]

Naja aber unbenutzer Code, der niemals zur Ausführung kommt sollte 
mindestens ein Warning bringen und nachträglich vom Compiler, sobald 
irgendeine Optimierung drin ist, wegoptimiert werden.

[(prx) A. K. (prx)]

Sowas wirft GCC auch ganz ohne eingeschaltete Optimierung raus.

[(prx) A. K. (prx)]

Troll schrieb:
> Naja aber unbenutzer Code, der niemals zur Ausführung kommt sollte
> mindestens ein Warning bringen

Lieber nicht, sonst wirkt man einer sauberen Programmierung entgehen. 
Denn obiger Code ist weit lesbarer, als der Ersatzcode mit 
Präprozessor-#if statt C-if.

GCC liefert daher auch mit -Wall keine Warnung.

[Klaus (Gast)]

A. K. schrieb:
> Lieber nicht, sonst wirkt man einer sauberen Programmierung entgehen.
> Denn obiger Code ist weit lesbarer, als der Ersatzcode mit
> Präprozessor-#if statt C-if.
>
> GCC liefert daher auch mit -Wall keine Warnung.

Ist ja auch klar. Denn der COde ist erreichbar. Je nach Wert von FCPU. 
Nicht erreichbarer Code ist per Definition Code der niemals erreicht 
werden kann. Also nicht mehr Wert als toter Zeichensalat. Dann, und nur 
dann, schmeißt der GCC eine Warnung, wenn er so eine Codestelle erkannt 
hat.

[(prx) A. K. (prx)]

Klaus schrieb:
> Ist ja auch klar. Denn der COde ist erreichbar.

Ist so ein Code auch erreichbar?

if (0) {
   g();
}

[Coder (Gast)]

In diesem Zusammenhang sind bzw. waren static und const gute Gehilfen.

[Klaus (Gast)]

A. K. schrieb:
> Ist so ein Code auch erreichbar?
>
> if (0) {
>    g();
> }

Entschuldige bitte, ich war etwas zu schnell mit meinen Gedanken. Du 
hast natürlich Recht. Denn der GCC bekommt das ja mit eingesetzter 
Konstante (nach dem Präprozessor) geliefert. Damit ist das Beispiel mit 
FCPU das gleiche wie if (0) .

Warum gibts denn da keine Warnung vom GCC? ;-)

[Christian B. (casandro)]

Naja in dem Beispiel ist es ganz einfach.
Aus dem Preprozessor kommt

  if (16000000UL == 8000000UL){
  ...
  }

  if (16000000UL == 16000000UL){
  ...
  }

raus, was im nächsten Schritt zu 0 und 1 zusammen gefasst wird, und die 
ifs fliegen raus.

Was anderes wäre es, wenn Du eine globale Variable hast, denn da gibt es 
für den Compiler keine sichere Methode festzustellen, ob die Verändert 
wurde oder nicht. (da gibts aber auch Möglichkeiten dem Compiler zu 
sagen was Du willst)

[(prx) A. K. (prx)]

Klaus schrieb:
> Warum gibts denn da keine Warnung vom GCC? ;-)

Weil es keinen Sinn ergibt, die Leute bei völlig normalem Code in 
sinnlosen Warnungen zu ersäufen.

Denk auch an Fälle wie
  inline void f(int x) { if (x) { ... } ... }
  void g(void) { f(0); }
Der Compiler sieht da if(0) mit und if(?) ohne Inlining.