Skip to content

Add dependent resource inference#6

Open
dcao wants to merge 39 commits intomasterfrom
dependent-res-infer
Open

Add dependent resource inference#6
dcao wants to merge 39 commits intomasterfrom
dependent-res-infer

Conversation

@dcao
Copy link
Collaborator

@dcao dcao commented Sep 2, 2020

dcao and others added 30 commits August 9, 2020 01:38
as of right now, this fails because of a sort error
we're also at risk of derefencing freed z3 asts right now
because we were including freed AST nodes, Z3 was throwing us errors.
resource constraints still can't be solved tho - this is another
bug
RAML-Queue and RAML-BitVec both work now :)
previously, nested abs potls would not have potential vars
created for them; this has been fixed
more of them work now :))
we basically throw our formulas at the internal cegis solver
and get the values of those formulas afterwards; no actual
optimizing is currently done, partially for performance
reasons
right now we just pass in the previous function arguments
as arguments; this may be expanded later on
- the z3 solve and optimize fns now take lists of formulas;
  the solve fn debug prints the unsat core on failure
- actual cegis optimization machinery is disabled just for
  testing
- this adds well-formedness fixes into this branch
- also fixes test script to run with correct args
we were only propagating our inferred potls in the inferredRVars
part of the state; we also have to do this in the resourceVars
part
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants