µChip XC16 hat Probleme bei Inline Assembler Befehlen

[Thomas S. (daimonion)]

Hallo Leute


Ich habe hier einen codepart den ich vom C30 Compiler auf den XC16 
Compiler portieren will. Das funktioniert auch alles super, bis auf ein 
Stück Code wo der Compiler mit aktivierter Optimierung aus dem Tritt 
gerät. Es handelt sich grob gesagt um folgendes Stück Code:

extern sint16 Math_CalcSinTaylor(uint16 phiX_u16)
{
    register sint16 AkkuA_s16 asm("A");
    register sint16 AkkuB_s16 asm("B");
    ...
    ...
    ...
    /* x^2 = x*x */
    AkkuB_s16 = __builtin_mpy(phiX_s16, phiX_s16, 0, 0, 0, 0, 0, 0);
    phiX2_s16 = __builtin_sac(AkkuB_s16, 0); /* store */
    ...
    ...
    ...
}

Die Fehlermeldung bei aktivierter Optimierungsstufe (egal welche) 
lautet:

../source/Core/HAL/Math.c: In function 'Math_CalcSinTaylor':
../source/Core/HAL/Math.c:401:1: error: Argument 0 should be an accumulator register (__builtin_sacr)
../source/Core/HAL/Math.c:401:1: error: Expected DSP accumulator result (__builtin_mac)

Daraus schliesse ich, dass der Compiler die Definition der Variablen 
AkkuA_s16 und AkkuB_s16 wegschmeisst.

Wie kann ich das denn verhindern?

Grüße
Daimonion

[Bronco (Gast)]

Geraten:
Vielleicht mußt Du die DSP-Register beim XC16 jetzt anders definieren 
als beim C30?

Oder vielleicht hilft auch schon ein volatile?

extern sint16 Math_CalcSinTaylor(uint16 phiX_u16)
{
    volatile register sint16 AkkuA_s16 asm("A");
    volatile register sint16 AkkuB_s16 asm("B");
    ...
}

[Thomas S. (daimonion)]

Danke für deine Antwort.

Leider konnte ich in den Style Guides hinsichtlich der Definition 
keinerlei Unterschied zwischen C30 und XC16 finden. Daher gehe ich erst 
mal davon aus, dass sich da nichts geändert hat.

Unseren Microchipmann habe ich dahingehend auch schon darauf angesetzt 
aber bisher konnte er mir noch nichts sagen.

volatile habe ich probiert. Dann compiliert er zwar, bringt aber ein 
warning dass die Variablen wahrscheinlich wegoptimiert werden könnten. 
Ach ne... ;)

Grüße
Daimonion

[Bronco (Gast)]

Könnte es daran liegen, dass die Akkus 24Bit breit sind und Deine 
Variablen nur 16Bit? Macht aber auch keinen Sinn...

[jonas biensack (Gast)]

Laut Fehlermeldung gibt __builtin_sacr einen Fehler nicht __builtin_sac?
Hm?

Gruß Jonas

[Thomas S. (daimonion)]

Hallo Leute

Ich konnte das Problem momentan damit lösen, dass ich die 
Variablendefinition nicht, wie dargestellt, in die Funktion schreibe, 
sondern quasi global in das C-File.

Ist zwar nicht schön, aber somit kann ich erst mal weiter den Code 
testen.