Skip to content

Enable -mllvm -verify-machineinstrs option by default #19

Description

@atrosinenko

Initially, it was proposed to enable -mllvm -verify-machineinstrs option by default in #15:

This option is useful for testing, but it would fail the build unless the LLVM_BRANCH specified includes llvm/llvm-project@22968f5, thus I marked this PR as a draft until we update the default branches as well.

Unfortunately, it turned out other changes to the backend made this check fail even with llvm/llvm-project@22968f5. As #15 introduces other important changes, I would rather investigate this failure in the background and unblock #15.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions