Scheduling enclaves in the C target #1872
Conversation
…ion between delay and forward body
core/src/main/java/org/lflang/generator/ReactionInstanceGraph.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/lflang/generator/c/CEnclaveGenerator.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/lflang/generator/c/CEnclaveGenerator.java
Outdated
Show resolved
Hide resolved
| import org.lflang.generator.RuntimeRange; | ||
| import org.lflang.generator.SendRange; | ||
|
|
||
| public class CEnclaveGraph { |
There was a problem hiding this comment.
Why doesn't this class use the existing graph classes?
There was a problem hiding this comment.
I was not able to get what I want from the Directed Graph class as it is only able to represent Nodes of arbitrary types not Edges. For the Enclave graph it is important to annotate the edges with after-delays and connection type (physical or normal)
|
Thanks for feedback Edward. I will address this. The PR is still a draft and I think I should do a proper pass through it and fix documentation and etc before anyone looks at it seriously |
Co-authored-by: Edward A. Lee <[email protected]>
…ocal RTI header files.
|
@lhstrh and @edwardalee, I am tagging you for a review now. We still have a test failure on macOS with coordinated shutdown of enclaves which I will look at in ~10 days. I have made a pass over all the changes and tried to bring the docs up to date. I kept my custom graph implementation due to the need to annotate edges, not nodes as is done in the other graphs. I will be mostly offline the next 8 days. |
Fix expensive way of finding the parent enclave of a reactorInstance
|
Closing this as we are pursuing a different AST strategy. See #2104 |
This PR adds the first version of the implementation of scheduling enclaves in the C target. It is paired with this PR in reactor-c: lf-lang/reactor-c#242