Open molenzwiebel opened 2 years ago
To quickly react to your last points involving the concurrent solver: When your specification semantically works with the concurrent solver, and there is no type-directed transformation, I think it is fine to use the traditional solver for now, and switch to the concurrent solver later. I'll add a TODO to that PR.
Exactly 11 months ago, I posted a comment about enabling the concurrent solver. I don't remember the discussion fully anymore, but the referred PR has landed, so technically, enabling the concurrent solver might be possible.
This PR contains the following changes:
tim
/dynamix
/dynamix_runtime
spoofax2 projects use source imports to depend on each other.Notes/issues that would be nice for you to take a look at:
ReadDynamixSpecTaskDef
is possibly a bit over the top and could be changed to just passing aResourcePath
, and having the reading be done in the dynamix runtime instead, but I wasn't entirely sure if that would work properly with PIE.createDynamixRuntimeConfig
inExecuteDynamixSpecificationTaskDef.java.mustache
. This works, but there might be a more idiomatic approach?dynamix_runtime
"language" is currently in the metalang folder, but it might be more appropriate to put it elsewhere (next to rv32im?).tim
stratego interpreter is currently bundled with the metalang (and not split intotim_runtime
or something similar). Currently done because it works and as mentioned this shouldn't be the long-term solution for running tim files.ESV
files don't seem to work. Try uncommenting thetim/colors
import in the dynamix Main.esv.0.1.0-SNAPSHOT
. Is that fine?Other notes/TODOs: