Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enums MVP. #819

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Enums MVP. #819

wants to merge 2 commits into from
+592 −134

Commits on Dec 7, 2021

  1. Enums MVP. 

    Supported cases:
- non-repr-C enums without non-null optimization;
- repr-C enums that are dataless / have the same layout for every case
- enums with non-null optimization that don't use pointers (e.g. Option<NonZeroUsize>).

Future work directions:
- Option<&_> and such - currently emits invalid SPIR-V, likely requires adaptations in mem2reg
- repr(C) enums that can be represented as (discriminant, union) - some of the groundwork is already there
    @ElectronicRU
    ElectronicRU committed Dec 7, 2021
    Configuration menu
    Copy the full SHA
    eb18850 View commit details
    Browse the repository at this point in the history

  2. Make Option<bool> also work while we're at it.

    @ElectronicRU
    ElectronicRU committed Dec 7, 2021
    Configuration menu
    Copy the full SHA
    a36fb0f View commit details
    Browse the repository at this point in the history