-
Notifications
You must be signed in to change notification settings - Fork 13k
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
regression when compiling substrate node with -Cinstrument-coverage
#117211
Labels
A-code-coverage
Area: Source-based code coverage (-Cinstrument-coverage)
C-bug
Category: This is a bug.
Comments
rustbot
added
the
needs-triage
This issue may need triage. Remove it if it has been sufficiently triaged.
label
Oct 26, 2023
I don't know how to make a minimal example to reproduce, but I can give some tips: // from https://github.com/gear-tech/gear/blob/70f599c530f6a0bfd0a1627d2c45d7a8d77bc3ad/runtime/vara/src/lib.rs#L275
impl pallet_grandpa::Config for Runtime {
type RuntimeEvent = RuntimeEvent;
type WeightInfo = ();
type MaxAuthorities = MaxAuthorities;
type MaxSetIdSessionEntries = MaxSetIdSessionEntries;
type KeyOwnerProof = <Historical as KeyOwnerProofSystem<(KeyTypeId, GrandpaId)>>::Proof;
type EquivocationReportSystem =
pallet_grandpa::EquivocationReportSystem<Self, Offences, Historical, ReportLongevity>;
}
// from https://github.com/paritytech/substrate/blob/033d4e86cc7eff0066cd376b9375f815761d653c/frame/grandpa/src/equivocation.rs#L174C5-L174C21
impl<T, R, P, L>
OffenceReportSystem<...>
fn process_evidence(
reporter: Option<T::AccountId>,
evidence: (EquivocationProof<T::Hash, BlockNumberFor<T>>, T::KeyOwnerProof), // <- error
) -> Result<(), DispatchError> {
...
}
} |
@rustbot label a-code-coverage |
rustbot
added
the
A-code-coverage
Area: Source-based code coverage (-Cinstrument-coverage)
label
Oct 26, 2023
saethlin
removed
the
needs-triage
This issue may need triage. Remove it if it has been sufficiently triaged.
label
Oct 31, 2023
2 tasks
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 8, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 8, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 22, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 22, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 22, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 22, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 25, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 25, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
atodorov
added a commit
to gluwa/creditcoin3
that referenced
this issue
Mar 26, 2024
cargo llvm-cov was hitting this issue: error[E0275]: overflow evaluating the requirement `<Runtime as pallet_grandpa::Config>::KeyOwnerProof == _` which is documented upstream in rust-lang/rust#117211 paritytech/polkadot-sdk#2641 and fixed in polkadot-sdk v1.6.0 and later, see paritytech/polkadot-sdk#2644
i3ima
added a commit
to sora-xor/sora2-network
that referenced
this issue
Aug 26, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-code-coverage
Area: Source-based code coverage (-Cinstrument-coverage)
C-bug
Category: This is a bug.
We just updated from
nightly-2023-04-25
tonightly-2023-10-14
and we are gettingAccording to backtrace, it seems the error appeared in #115801
Suspect line:
rust/compiler/rustc_traits/src/normalize_projection_ty.rs
Lines 57 to 64 in 104ac7b
I tried this code:
I expected to see this happen: code compiles with and without the
-Cinstrument-coverage
option.Instead, this happened: error
E0275
with-Cinstrument-coverage
Meta
rustc --version --verbose
:Backtrace
The text was updated successfully, but these errors were encountered: