Skip to content

Commit

Permalink
Rollup merge of #130690 - RalfJung:interpret-abi-compat-fixme, r=jiey…
Browse files Browse the repository at this point in the history
…ouxu

interpret: remove outdated FIXME

The rule about `repr(C)` types with compatible fields got removed from the ABI compat docs before they landed, so this FIXME here is no longer correct. (So this is basically a follow-up to #130185, doing some more cleanup around deciding not to guarantee ABI compatibility for structurally compatible `repr(C)` types.)
  • Loading branch information
GuillaumeGomez authored Sep 22, 2024
2 parents 82b4177 + 27400ea commit f40efa4
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion compiler/rustc_const_eval/src/interpret/call.rs
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,6 @@ impl<'tcx, M: Machine<'tcx>> InterpCx<'tcx, M> {
}

// Fall back to exact equality.
// FIXME: We are missing the rules for "repr(C) wrapping compatible types".
Ok(caller == callee)
}

Expand Down

0 comments on commit f40efa4

Please sign in to comment.