150Fermer152
Kevin KoflerLe 17/11/2008 à 09:56
@squalyl:
Bah, je ne comprends pas pourquoi c'est à moi que tu t'en prends. PedroM n'a pas à utiliser des fonctionnalités qui n'existent pas dans la version officielle la plus récente de la chaîne d'outils (un problème qu'on retrouve d'ailleurs dans la version en développement actuellement, sauf que cette fois-ci c'est ld-tigcc qu'il forke, pas A68k), surtout s'il est trivial de faire sans (se passer du CNOP:
+START_ARCHIVE EQU ROM_BASE+$30000	; Start of the archive!
[...]
-	CNOP 0,65536			; Skip the last bytes of this sector
-START_ARCHIVE:				; Start of the archive!

aurait été moins de travail que faire le patch pour A68k) et si le patch pour la chaîne d'outils n'a pas été accepté (il ne m'a même pas demandé mon avis avant de l'utiliser, et quand il m'a envoyé le patch et que je l'ai rejeté, il a refusé de modifier PedroM pour assembler sans le patch, alors que ça aurait été trivial de le faire, cf. ci-dessus).

Pour les nouvelles fonctionnalités qu'il a rajoutées à ld-tigcc, il serait aussi tout à fait possible de s'en passer:
* pour les BSS, on pourrait utiliser le système du header Vars.h comme dans l'ancienne version - ça permettrait aussi d'utiliser les adressages en .w systématiquement (même pour les opérandes de destination), ce que la version actuelle du patch BSS ne fait pas (c'est la raison principale pour laquelle je n'accepte pas ce patch en l'état actuel),
* __ld_first_user_sector (nom non définitif) pourrait facilement être codé en dur (même histoire que pour le CNOP dans l'ancienne version),
* __ld_bss_even_end (nom non définitif) pourrait facilement être calculé en temps d'exécution:
 move.l #__ld_bss_end,d0
 addq.l #1,d0
 lsr.w #1,d0
 add.w d0,d0