[AVR] compiler optimiert strings nicht richtig

> Was muss ich tun, damit der string nur einmal eingebunden wird?
Mach eine globale Variable mit dem String:

1

 char ERROR_SD_READ[] = "Error SD_read";

2

  :

3

    lcd_puts_P( ERROR_SD_READ );

4

  :

das macht keinen Sinn.
dann brauch der string sowohl daten, als auch programmspeicher + die 
Pointervariable
ich suche eher einen Compilerswitch oder trick, der ihn zwingt doppelte 
strings nur einmal anzulegen.

>Ein 'guter' Compiler macht das von alleine. Zur Not auf 'size'
>optimieren lassen.

hätt ich ja auch gedacht

ist halt ein gcc
duck und weg

und auf size optimieren ist schon an.

aber holgers variante scheint zu funktionieren.

Trotzdem hätt ich lieber ne Variante, wo das der compiler automatisch 
macht.

Worin liegt das Problem, den String nur einmal zu definieren, ihn aber 
mehrmals zu verwenden?

1

char ERROR_SD_READ[] PROGMEM = "Error SD_read";

2

...

3

lcd_puts_P( ERROR_SD_READ );

Und ja, der String ist dann nur einmal im Speicher (Flash).
Und nein, das verbraucht keinen zusätzlichen Platz für irgendwelche 
Pointer.

Ach übrigens: das funktioniert wegen dem zusätzlichen Attribut nicht. 
"Normale" Strings (also in RAM) legt er automatisch zusammen.

Jeder String ist ein eigenständiges Objekt. Ich wüsst nicht wie gcc das 
optimieren können sollte.

Das ist eine Aufgabe für ld. Er kann Strings mixen; dazu setzt man an 
der entsprechenden Section das S-Flag (mergeable strings).

Dazu muss jeder String in seine eigene Input-Section.

Johann

Johann L. schrieb:
> Jeder String ist ein eigenständiges Objekt. Ich wüsst nicht wie gcc das
> optimieren können sollte.

Es geht um Stringliterale gleichen Inhalts. Hier wird normalerweise der 
eigentliche String nur einmal angelegt. Das Section-Attribut hebelt das 
aber irgendwie aus, und der String wird mehrfach angelegt.

Wenn ich

1

const char STR1[] = "Hallo";

2

const char STR2[] = "Hallo";

mit avr-gcc 4.3.2 übersetze gibt das

1

.global  STR1

2

  .data

3

  .type  STR1, @object

4

  .size  STR1, 6

5

STR1:

6

  .string  "Hallo"

7

.global  STR2

8

  .type  STR2, @object

9

  .size  STR2, 6

10

STR2:

11

  .string  "Hallo"

12

.global __do_copy_data

Was willst du mir damit sagen? Natürlich sind STR1 und STR2 zwei 
unterschiedliche Objekte. Es geht nicht um Variablen, sondern die 
Literale.

1

Func1("Hallo");

2

Func2("Hallo");

An beide Funktionen wird der selbe Pointer übergeben.

genau nur scheint er acr-gcc das nicht hinzubekommen, wenn man die 
Strings in den Program-memory stecken will.
Dann legt er jeden extra an.

Aber Holgers lösung funktioniert.
Ist sowiso sauberer die messages an einer Stelle zu halten.

Stefan Ernst schrieb:
> Was willst du mir damit sagen? Natürlich sind STR1 und STR2 zwei
> unterschiedliche Objekte. Es geht nicht um Variablen, sondern die
> Literale.
>
>

1

> Func1("Hallo");

2

> Func2("Hallo");

3

>

> An beide Funktionen wird der selbe Pointer übergeben.

Du hast aber keine Literale:

1

#include <avr/pgmspace.h>

2

3

extern void foo1 (const char*);

4

extern void foo2 (const char*);

5

6

void foo (void)

7

{

8

    foo1 (PSTR ("hallo"));

9

    foo2 (PSTR ("hallo"));

10

}

Wird expandiert zu

1

void foo (void)

2

{

3

    foo1 ((__extension__({static char __c[] __attribute__((__progmem__)) = ("hallo"); &__c[0];})));

4

    foo2 ((__extension__({static char __c[] __attribute__((__progmem__)) = ("hallo"); &__c[0];})));

5

}

und damit gibt es zwei __c-Objekte, eines für jedes "hallo".

Johann