search

cdisselkoen / llvm-ir

LLVM IR in natural Rust data structures
MIT License
550 stars 46 forks source link

Add Github Actions CI (v2) #51

Closed Benjins closed 11 months ago

Benjins commented 11 months ago

This was originally added by @kaoet in #40 . However, they apparently ran into test failures for it. The detailed logs for the CI job from that PR has since expired, so I can't be sure, but it is possibly the issue seen in #44 where tests in debug hit a SIGILL from a bug in LLVMGetOrdering

I re-based the PR, and made the tests run with --release, which works around the issue. I also added llvm-16 to the matrix, since we support that now

You can see the successful CI run here: https://github.com/Benjins/llvm-ir/actions/runs/7071283452

There are some warnings (mostly from unused doc comments in module.rs), but no errors/test failures

cdisselkoen commented 11 months ago

Thanks, looks good. Having CI on this repo will be great