Skip to content

replace type_use with LLVM's mergefunc pass #9222

Closed
@thestinger

Description

@thestinger

The type_use implementation is buggy, leading to nightmarish code generation errors. The mergefunc pass will be strictly better in terms of code size reduction since it compares equality of the function bytecode after normalizing it. I think it will catch nearly every valid case we currently do with type_use, if not all.

We should drop the maintenance burden as soon as we get mergefunc working (or sooner) and revisit it as a potential compile-time optimization in the future.

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-codegenArea: Code generationC-cleanupCategory: PRs that clean code up or issues documenting cleanup.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions