Skip to content

CtxRed needs to expand more type synonyms #311

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
quark17 opened this issue Feb 3, 2021 · 0 comments
Open

CtxRed needs to expand more type synonyms #311

quark17 opened this issue Feb 3, 2021 · 0 comments
Labels
pre-github Transferred from Bluespec Inc's bug database

Comments

@quark17
Copy link
Collaborator

quark17 commented Feb 3, 2021

As mentioned in #310, a call to expandSyn in CtxRed had to be removed because of issues with TLM libraries. This code:

do -- XXX disable expanding of type synonyms until
   -- XXX failures with TLM instances are resolved
   -- XXX (vqs_extra, t1) <- expPrimTCons t0 (expandSyn t0)
   (vqs_extra, t1) <- expPrimTCons t0

should be replaced with this:

do (vqs_extra, t1) <- expPrimTCons (expandSyn t0)

An example can be found at bsc.typechecker/primtcons/ExpSizeOf_InstancesBaseSyn.bsv. Without expandSyn, this example fails to resolve a context:

Error: "ExpSizeOf_InstancesBaseSyn.bsv", line 6, column 10: (T0031)
  The provisos for this expression could not be resolved because there are no
  instances of the form:
    ExpSizeOf_InstancesBaseSyn::C#(ExpSizeOf_InstancesBaseSyn::S, 12)

This is bug 1729 in Bluespec Inc's pre-GitHub bug database.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pre-github Transferred from Bluespec Inc's bug database
Projects
None yet
Development

No branches or pull requests

1 participant