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

Bump CIRC, fix CI build, add lifetime annotations #252

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Bump CIRC, fix CI build, add lifetime annotations #252

wants to merge 6 commits into from

Commits on Jun 1, 2023

  1. Bump CIRCT

    @maerhart
    maerhart committed Jun 1, 2023
    Configuration menu
    Copy the full SHA
    37f9596 View commit details
    Browse the repository at this point in the history

Commits on Sep 30, 2023

  1. Add missing libraries 

    Build now works with rust 1.63 (when using backtrace = "=0.3.68")
    @stefanlippuner
    stefanlippuner committed Sep 30, 2023
    Configuration menu
    Copy the full SHA
    8fd4e59 View commit details
    Browse the repository at this point in the history

  2. Use ninja to build llvm and circt, pin Ubuntu version in CI 

    Uing makefiles and parallel build seems to cause errors in CI.
    @stefanlippuner
    stefanlippuner committed Sep 30, 2023
    Configuration menu
    Copy the full SHA
    3ece4eb View commit details
    Browse the repository at this point in the history

  3. Add missing lifetime annotations for trait objects 

    This caused compiler errors for rust >= 1.65
    @stefanlippuner
    stefanlippuner committed Sep 30, 2023
    Configuration menu
    Copy the full SHA
    8c4fbe9 View commit details
    Browse the repository at this point in the history

  4. Pin Rust toolchain version to v1.66 

    Newer toolchains cause stack overflow errors in some tests. That may be related to
a memory alignment change in Rust v1.67 (rust-lang/rust#102750)
    @stefanlippuner
    stefanlippuner committed Sep 30, 2023
    Configuration menu
    Copy the full SHA
    a825813 View commit details
    Browse the repository at this point in the history

  5. test: Ignore error in common_cells (broken previously), remove old IG… 

    …NOREs

The common_cells test was already failing as of 37f9596

The removed ignores referenced issues that have been closed and the tests
pass now.
    @stefanlippuner
    stefanlippuner committed Sep 30, 2023
    Configuration menu
    Copy the full SHA
    e92cd9b View commit details
    Browse the repository at this point in the history