todo.org

Regressions introduites par Pascal

Ya un probleme avec ce fichier lustre (compilait avant)

• State “WAITING” from “STARTED” [2013-01-17 Thu 10:48]
• State “STARTED” from “TODO” [2013-01-17 Thu 10:48]

file:test/should_work/call/call04.lus

il semble y avoir une confusion entre parametre et arguments lors d’appels de noeuds définis vie des itérateurs de tableau

idem pour file:test/should_work/fab_test/morel2.lus et pleins d’autres. hm, y’aurait pas d’itérateurs dans celui la.

-> ok : c’était à c’était cause de l2lAliasType qui faisait que les types lic n’étaient plus uniques et du coups les substitutions dans l’expansion des noeuds ne se faisaient plus correctement.

je met en attente en attendant de savoir ce qu’on fait de ce module. moi j’ai bien envie de virer AbstractType de Lic.type_. En effet, j’avais fais attention à tous les virer pour éviter les soucis, mais le plus propre c’est d’y virer vraiment.

Pascal lui, s’en ait servir pour faire des types alias, alors que ca n’est pas fait pour. Cela dit, si on créé des types alias, on risque d’avoir le memegenre de soucis. A quoi ca sert d’avoir de tels types ? pour moi le role de la compil ca serait plutot de les virer que de les rajouter, mais bon. A discuter. cf point d’apres

Enlever Abstract_type_eff de Lic.type_ ou vérifier partout que c’est correct.

• State “TODO” from “” [2012-12-20 Thu 17:26]

dans lic.ml, on definit les types compilés ainsi : and type_ =

Bool_type_eff

Int_type_eff

Real_type_eff

Abstract_type_eff of Ident.long * type_

Array_type_eff of type_ * int

…

Mais est-ce vraiment utile de garder le Abstract_type_eff à ce niveau ?

en effet, ca oblige à traiter les 2 cas en permanence (par ex lors des transfo llic2lic).

Pascal suggerer carrément de

• definir un type type_ref of string
• transformer les expressions lic de telle sorte que il n’y ait plus de type_ mais juste des type_ref

Car en fait, actuellement, le type Lic.type_ sert à faire la verif de type et a representer le type des expressions lic. Du coup le type des expressions est inutilement compliqué; d’ou l’idée d’avoir juste des “type_ref of string” (Ce qui simplifiera la travail des passes ultérieures, style lic2c).

Bon, je ferai ca quand tous les tests fonctionneront et pendant que j’essairais de me passer de ulglyStuff/id_solcer

Refaire marcher les tests de non-reg qui sont cassés

suites aux modifs de Pascal de l’été 2012

• State “TODO” from “” [2012-10-26 Fri 14:59]

En fait, ce qui est cassé, c’est surtout le -ec puisqu’il n’a pas rebranché l’expansion de tableau et de noeud.

il a fait

• mv tests _tests
• mkdir tests
• cp _tests/Makefile tests
• créer des tests qu’il a mis dans tests…

Il ne detecte plus les erreurs d’instanciation de modeles

• State “TODO” from “” [2012-12-21 Fri 11:08]

cd src; ../objlinux/lus2lic -vl 2 --nonreg-test should_fail/type/parametric_node2.lus

file:test/should_fail/type/parametric_node2.lus

internal error in l2lExpandArrays.ml with -esa

• State “TODO” from “” [2013-01-18 Fri 10:48]

./lus2lic -esa -o /tmp/ply02.ec should_work/broken/ply02.lus

oops: lus2lic internal error File “objlinux/l2lExpandArrays.ml”, line 213, column 14 when compiling lustre program file:test/should_work/broken/ply02.lus

nb : le mode -en ne marche pas bas bien non plus…

./lus2lic -en -o /tmp/ply02.ec should_work/broken/ply02.lus Error in file “/home/jahier/lus2lic/test/should_work/broken/ply02.lus”, line 8, col 2 to 2, token ‘o’: Variable o is already defined.

mauvais numero de ligne lors d’erreur d’instanciation de parametres de noeud

• State “TODO” from “” [2012-12-21 Fri 10:58]

cd src; ../objlinux/lus2lic -vl 2 --nonreg-test should_fail/type/parametric_node.lus

Opening file /home/jahier/lus2lic/test/should_fail/type/parametric_node.lus Error in file “parametric_node.lus”, line 4, col 17 to 17, token ‘n’: can’t eval type: bad array size, int expected but get real

le message serait meilleur s’il indiquait la ligne 17 où l’erreur est effectuée

file:src/astInstanciateModel.ml file:test/should_fail/type/parametric_node.lus

Pas dur

Mettre le sha dans l’affichage de –version

car c’est plus facile dans git pour retrouver ses petits

• State “TODO” from “” [2012-10-26 Fri 14:59]

Intégrer le résultat de mly2bnf dans le manuel

Nommage des variables fraiches : Reprendre LicVarName.ml

• State “TODO” from “” [2013-01-16 Wed 18:03]

car c’est completement n’importe quoi (j’ai réussit a faire des clashs entre les var inventés dans split et dans expandnodes !!)

Pascal a introduit un mecanisme qui shunte LicName -> en discuter avec lui.

A faire

fix unresolved tests (timeout -> performance bugs)

• State “TODO” from “” [2013-01-11 Fri 11:04]

par ex, file:./test/should_work/ec.lus prend un temps infini alors qu’il n’est pas si gros (y’a 500 variables, mais bon). bon, en fait il prend 26 seconds, ce qui n’est pas infini, mais bien long tout de meme.

nb: c’était deja le cas avant les changements de Pascal.

cf file:test/lus2lic.gprof 69% du temps est passé dans unify clock !!!!!

J’ai l’impression que c’est lié au fait que ce programme ne definit que des contantes. Or les constantes sont potentiellement sur n’importe quelle horloge, ce qui fait que l’algo manipule un gros paquet de ‘clock_var of int’ et que l’on passe beaucoup de temps à faire des apply_substs2

cf file:test/perf/ contenant les resultats de gprof et ocamlprof sur ec.lus

les (nouveaux) tests ne capturent pas les changements de # lignes dans les should_fail

• State “TODO” from “” [2013-01-11 Fri 11:15]

Définir les fonctions de UglyStuff proprement

• State “TODO” from “” [2012-12-10 Mon 16:38]

file:~/lus2lic/src/uglyStuff.ml

Rajouter le with à la caml pour les structures

• State “TODO” from “” [2012-10-26 Fri 14:59]

Dans file:src/eff.ml::195 on pourrait virer la notion de call by name

qui ne sert que pour les structures. Mais au niveau du Eff, on pourrait s’en être débarrassé au profit d’un appel par position. A faire au moment de la compilation des expressions.

question : est-ce que cela nous prive d’une optim pour le traitement du with ? comment fait caml ?

• State “TODO” from “” [2012-10-26 Fri 14:59]

on devrait se passer de ‘static_arg list’ pour le champ PREDEF_CALL

(c’est censé marcher) cf file:./src/eff.ml::206

• State “TODO” from “” [2012-10-26 Fri 14:59]

L’ideal serait de se passer du PREDEF_CALL (et de passer par le CALL normal)

• State “TODO” from “” [2012-10-26 Fri 14:59]

le Eff.WITH (aka if statique) n’a pas lieu d’être !

y virer !!

• State “TODO” from “” [2012-10-26 Fri 14:59]

Vieux trucs (?)

operateurs iterables

• State “TODO” from “” [2012-03-30 Fri 17:03]

• mettre dans la doc
• voir si on ne pourrait pas completer la liste

en mettant tous les operateurs unaires de file:~/lus2lic/src/syntaxTreeCore.ml::91 dans file:~/lus2lic/src/predef.ml::62

• tout au moins, eviter les assert false sur

Revoir le nommage des instances de noeuds parametriques

par ex, pour should_work/NONREG/param_struct.lus ca invente des noms bien débiles du point de vue du nom du pack. Style ‘mk_tab__param_struct::toto_toto_3’ alors qu’aucun package ne s’appelle ‘mk_tab__param_struct’

Changer la facon de numeroter les versions dans lus2lic (faire comme dans lurette)

[2010-11-08 Mon]

Verifier les boucles combinatoires quand on genere du ec (car ec2c ne fait pas la verif !)

[2010-06-07 Mon]

Refaire une passe pour virer une fois pour toute cette histoire d’idref dans Eff.

cf file:/~/lus2lic/src/eff.ml line 189 QU: Pascal l’a fait ?

lic2c : le jour ou on genere du code C, y’a peut-etre des trucs a recuperer

• State “TODO” from “” [2012-12-10 Mon 14:32]

des trucs chez cedric pasteur (thesard marc p) qui a une implementation pour optimiser la maj des tableaux