Check cranelift when using the cargo-clif executable

[frederikhors]

Fixes #537 (comment).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

[bjorn3]

This may be a bit too loose. It would also match for example RUSTFLAGS="--cfg use_cranelift_for_wasm" or something like that. Not that I could find any actual cases on github where that happens though.

[justsmth]

From my testing, I believe most platforms were fine linking using the "no u1" (terrible name) bindings, even if it's not required. So it might be ok for this to be overly broad in catching "cranelift".

Would this also catch for consumers using codegen-backend = "cranelift" in their .cargo/config.toml?

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.62%. Comparing base (c358484) to head (a466e16).
⚠️ Report is 337 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1047      +/-   ##
==========================================
- Coverage   95.80%   92.62%   -3.18%     
==========================================
  Files          61       71      +10     
  Lines        8143    10053    +1910     
  Branches        0    10053   +10053     
==========================================
+ Hits         7801     9312    +1511     
- Misses        342      450     +108     
- Partials        0      291     +291     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[frederikhors]

Would this also catch for consumers using codegen-backend = "cranelift" in their .cargo/config.toml?

I think this already works in the actual main branch code.

[frederikhors]

I can't wait to try it!

[justsmth]

Hi @frederikhors -- We have a CI job that verifies the following text appears in the PR description:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

If you agree, would you update the PR description with this text?

[frederikhors]

Do you mean in the first post? Done.