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.

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

1.54.0 new test assembly/asm/bpf-types.rs fails on all architectures on Debian rustc build #89689

Closed
infinity0 opened this issue Oct 8, 2021 · 2 comments · Fixed by #89697
Closed
Labels
C-bug Category: This is a bug.

Comments

@infinity0
Copy link
Contributor

The error is pretty straightforward

stderr:
------------------------------------------
warning: unused macro definition
  --> /<<PKGBUILDDIR>>/src/test/assembly/asm/bpf-types.rs:20:1
   |
20 | / macro_rules! stringify {
21 | |     () => {};
22 | | }
   | |_^
   |
   = note: `#[warn(unused_macros)]` on by default

error: couldn't allocate output register for constraint 'w'
  --> /<<PKGBUILDDIR>>/src/test/assembly/asm/bpf-types.rs:42:19
   |
42 |             asm!("{} = {}", out($class) y, in($class) x);
   |                   ^

error: aborting due to previous error; 1 warning emitted

We don't use rust's LLVM but a more vanilla LLVM in the Debian rust build, could that be the issue?

CC @alessandrod as the author of the test.

I will just disable the test for now in Debian.

@infinity0 infinity0 added the C-bug Category: This is a bug. label Oct 8, 2021
@alessandrod
Copy link
Contributor

We don't use rust's LLVM but a more vanilla LLVM in the Debian rust build, could that be the issue?

I'm guessing that Debian LLVM is missing https://reviews.llvm.org/D102118

@infinity0
Copy link
Contributor Author

That was from 5 months ago, which version of LLVM was it released in?

The test says // min-llvm-version: 10.0.1 so this looks like it needs fixing.

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Oct 9, 2021
Fix min LLVM version for bpf-types test

The test requires https://reviews.llvm.org/D102118 which was released in LLVM 13.

Closes rust-lang#89689
@bors bors closed this as completed in 8683d36 Oct 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants