Récupérer un nombre - Page 3

A tout hasard, n'y a-t-il pas un switch qui aligne les données automatiquement ? Pourrais-tu poster tes options de compilation ?
Sinon, à moins qu'il y ait quelque chose que je ne sache pas, ça ressemble à un bug.

> Y'a encore bcp de risques avec db92?
Il y en a bien moins qu'avant. Je ne l'ai pas downloadé récemment, mais c'est au moins presque stable (pas testé, je ne peux pas dire). Peu de bugs sont postés, en tout cas...

Les switchs de A68K sont: -g -t

geogeo
a écrit : Ce qui signifie que le compilateur met autmatiquement les octets sur une adresse paire donc ma liste devient:

C'est bizarre. Je n'ai jamais vu A68k faire ça auparavant.

Je voudrais bien qu'il ne fasse pas ça. Mais il le fait. Vous voulez que je poste tout?
Je sais pas comment faire, je ne peut plus avancer dans mon programme tout ce décale.

j'ai testé sous la 2.71.f2 de Fargo II, ça le fait aussi...

geogeo
a écrit : Je sais pas comment faire, je ne peut plus avancer dans mon programme tout ce décale.

Tu implémentes un switch qui dit à A68k de ne pas faire ça. Les sources sont là pour ça. Envoie-moi un patch quand tu as fini. Sinon, je peux essayer de m'en occuper, moi aussi, si tu veux, mais je n'ai pas vraiment le temps, donc ça pourrait devoir attendre 1 ou 2 semaines.

Elles sont où les sources? Je vais voir ça.

Dans les sources de TIGCC.

Ouha 7 mo à télécharger heuresement que j'ai l'ADSL, je vais voir ça. Pour le compiler je compile avec quoi comme logiciel? VC++ ça suffit?

C'est dans le fichier Adirect.c.
AddrCnt = AddrBndW (AddrCnt);

Le code qui nous intéresse est là :    case DC:						/* DC */
	if ((Size == Word) || (Size == Long))

Donc apparamment, A68k n'arrondit à l'adresse paire suivante que si Size représente un word ou un long. J'ai essayé de regarder rapidement où est modifié Size, mais je n'ai pas trouvé...

+x-OpCode, SizeErr); /* Invalid size */ Size = Word; /* Default to Word. */ break; } }

Je pense qu'une autre partie intéressante est dans le fichier Symtab.c, il s'agit de la fonction GetParts :    if ((x < OpCode) || (*x != '.'))	/* If no explicit size is given */
/*	Size = Word;	*/		/*  default to Word (16 bits).  */
/* bugfix by Kevin Kofler for the TIGCC team in v.2.71.F3b: */
    {
	Size = Word;			/*  default to Word (16 bits).  */
    noExplicitSize=TRUE; /* remember that no explicit size was given! */
    }

    else {
/* bugfix by Kevin Kofler for the TIGCC team in v.2.71.F3b: */
    noExplicitSize=FALSE; /* remember that an explicit size was given! */
    
	*x++ = '\0';			/* Chop off size extension. */
	switch (toupper(*x)) {
	case 'B':			/* Byte */
	case 'S':			/* Short Branch */
	    Size = Byte;
	    break;
	case 'W':			/* Word */
	    Size = Word;
	    break;
	case 'L':			/* Long */
	    Size = Long;
	    break;
	default:
	    Error (OpLoc

Désolé si c'est un peu long. En plus, je ne suis même pas sûr que ce soit bien cet endroit qui nous intéresse.
En tout cas, d'après ce morceau de code et celui du post précédent, il ne devrait pas y avoir d'alignement automatique sur une adresse paire lorsqu'on manipule un byte.

Bon, je vais faire des tests...

C'est le morceau du message #69 où se situe le problème. Le code écrit là veut dire: si le DC est en .W ou .L, aligne sur une adresse paire.

Ce qu'il faut faire maintenant est rajouter un switch en ligne de commande pour supprimer cela. (Je pense qu'il est une bonne idée de laisser le code actif par défaut parce qu'un dc.w à une adresse impaire n'est normalement pas une très bonne idée.)

Hum en fait, j'avais mal compris ton problème.
Oui, A68K rajoute un octet nul si ton word n'est pas à une adresse paire. Mes tests le montrent, ainsi que le code posté dans les deux posts précédents...
J'avais mal compris ton pb au début, je croyais que a68k alignait sur des adresses paires même tes bytes.

OK, pour faire ça, il faut rajouter une variable globale (beurk) qu'on initialise à 0 si le switch n'est pas actif, ou à 1 s'il l'est ?
Sinon, pour le switch on prend quelle lettre ? Il reste A, B, C, J ou U.
Pourquoi pas U comme UnEven ?

U comme Unaligned, c'est très bien.

Mais je pense qu'à ce point, je pourrai m'en sortir moi-même ce weekend. Ce n'est plus qu'une question d'insérer entre 3 et 6 lignes au bon endroit.

Ouai U c'est pas mal. Donc c'est un bug de A68K ou c'est vraiment fait comme ça?
jackiechan>Merci d'avoir pris du temps pour corriger ce problème.

C'est fait exprès, vu qu'il y a du code exprès pour le faire.

C'est pas logique je trouve.

c'est pas logique du point de vue de la définition des dc.x, mais du point de vue de l'utilisation d'un dc.w si... enfin le problème vient surtout du fait que ça n'avait pas l'air d'être spécifié... enfin c'est réglé maintenant

Bon, je ne sais pas comment on fait un PATCH, mais à mon avis, voici les modifications qu'il faudrait apporter à la source :

Fichier A68kglb.h :
Suppress automatic word-align */Ajouter à partir de la ligne 60 :GLOBAL int UnAligned; /* -u

Fichier A68kMain.c :
ligne 77 :UnAligned = Ajouter à la
se 'U': UnAligned = TRUE; break;Ajouter à partir de la ligne 224 :ca

Fichier Adirect.c :
Long)) && !UnAligned)Modifier la ligne 38 : if (((Size == Word) || (Size ==

Voilà, c'est en gros ce qu'il faut faire à mon avis pour que ça marche.
Mais en fait, j'ai remarqué que les autres switch touchent à une variable appelée cmderror, je pense que celui là aussi devra la modifier, mais comme je n'ai pas compris à quoi elle sert, je ne sais pas ce qu'il faut mettre.

kevin>Par contre si je veut diffuser A68K, c'est possible et si oui je doit diffuser la lience GPL c'est ça? Et signaler cette licence dans ma doc?

geogeo a écrit :
jackiechan>Merci d'avoir pris du temps pour corriger ce problème.

Ça a été un réel plaisir.

Bon voilà. Essaye cette version pour voir si elle marche mieux. (N'oublie pas de passer le switch -u. )

jackiechan a écrit :
Bon, je ne sais pas comment on fait un PATCH, mais à mon avis, voici les modifications qu'il faudrait apporter à la source :
[...]
Voilà, c'est en gros ce qu'il faut faire à mon avis pour que ça marche. Mais en fait, j'ai remarqué que les autres switch touchent à une variable appelée cmderror, je pense que celui là aussi devra la modifier, mais comme je n'ai pas compris à quoi elle sert, je ne sais pas ce qu'il faut mettre.

C'est à peu près ça, mais tu as raté quelques endroits à modifier (notamment la liste des switches dans A68kmain.c, et aussi les fichiers de documentation), et tu as oublié l'appel à checkswitch (c'est à ça que sert cmderror). Et j'ai déjà fait la correction avant d'avoir lu ce message, mais merci quand-même.

geogeo
a écrit : kevin>Par contre si je veut diffuser A68K, c'est possible et si oui je doit diffuser la lience GPL c'est ça? Et signaler cette licence dans ma doc?

Ce n'est pas sous GPL, mais la documentation dit:

     Note:  This program is Freely Distributable, as opposed to Public
Domain.  Permission is given to freely distribute this program provided no
fee is charged, and this documentation file is included with the program.

Donc tant que tu inclus la documentation et que tu ne demandes pas de l'argent pour la distribution, tu peux distributer A68k.

Merci, je viens de le tester et ça mache très bien.

OK, je vais envoyer ça à Sebastian pour la prochaine version de TIGCC (qui sera la 0.95 bêta 1, donc pas pour tout de suite), après avoir mis à jour la documentation au format .hss.

Dans combien de temps sortira à peu près la 0.95 b1 ?

Quand le nouveau linkeur sera en un état qui qualifie au moins de bêta (et pas d'alpha). On y est presque. Sebastian est en train d'écrire la documentation, qui est le dernier grand point manquant. Je vais aussi devoir faire 2-3 trucs sur GCC et GNU as pour qu'ils coopèrent mieux avec le nouveau linkeur (les fichiers objet existants seront quand-même utilisables, mais certaines nouvelles fonctionnalités du nouveau linkeur ne marcheront que si on recompile avec un GCC adapté).

PS: Quand on parle d'état de bêta, c'est pour les targets TI-89/92+/V200. Pour Fargo, ça sera complètement expérimental, et il faudra probablement récupérer les fichiers objet nécessaires pour créer le header binaire des programmes/librairies Fargo séparément (Je le mettrai évidemment à disposition.), vu que Sebastian préfère ne pas inclure les fichiers Fargo dans TIGCC avant d'avoir décidé de manière définitive comment on les consolidera avec le système de documentation et de headers existant. (Actuellement, il n'y a ni documentation, ni headers, ni librairie statique pour Fargo. J'ai tout déclaré moi-même pour Backgammon.)

Est-ce qu'il sera possible d'inclure faststr.h dans TIGCC 0.95 Beta 1, si ça avance suffisamment vite (cf le gros topic que je viens de poster à l'instant sur notre forum) ?