Skip to content

Remove TyTy::UnitType #286

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

Merged
merged 1 commit into from
Mar 27, 2021
Merged

Remove TyTy::UnitType #286

merged 1 commit into from
Mar 27, 2021

Conversation

dkm
Copy link
Member

@dkm dkm commented Mar 15, 2021

This is WIP for #252 and awaits copyright assignment.

Removed TyTy::UnitType and TyTy::TypeKind::UNIT.
Replaced by TyTy::TupleType with an empty list of fields.
Added default empty vector for fields in ctor for TyTy::TypeType.

@dkm dkm marked this pull request as draft March 15, 2021 20:08
@dkm dkm changed the title WIP for #252 Remove TyTy::UnitType Mar 15, 2021
@dkm dkm force-pushed the dev/252 branch 2 times, most recently from ea99b82 to ff5df15 Compare March 16, 2021 17:23
@philberty
Copy link
Member

Nice one, the code was simpler than i thought

@philberty philberty linked an issue Mar 18, 2021 that may be closed by this pull request
@dkm dkm force-pushed the dev/252 branch 3 times, most recently from 24d723d to e6beea6 Compare March 23, 2021 17:30
Removed TyTy::UnitType and TyTy::TypeKind::UNIT.
Replaced by TyTy::TupleType with an empty list of fields.

Added default empty vector for fields in ctor for TyTy::TypeType.
@dkm dkm marked this pull request as ready for review March 26, 2021 19:59
@philberty philberty merged commit fabb389 into Rust-GCC:master Mar 27, 2021
dkm added a commit to dkm/gccrs that referenced this pull request Mar 27, 2021
the PR Rust-GCC#302 and Rust-GCC#301 were conflicting because they are both modifying the tests.
Move the conflicting test and add correct dg-error directive.

continue1.rs is less verbose after Rust-GCC#286, remove the dg-excess-error.
@dkm dkm deleted the dev/252 branch March 27, 2021 20:18
dkm added a commit that referenced this pull request Mar 28, 2021
the PR #302 and #301 were conflicting because they are both modifying the tests.
Move the conflicting test and add correct dg-error directive.

continue1.rs is less verbose after #286, remove the dg-excess-error.
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.

Remove TyTy::UnitType
2 participants