Add a test for the new type_name intrinsic.#761
Conversation
type_name.type_name.
|
I'd prefer to wait for rust-lang/rust#61399 to get merged and then remove the impl here |
|
Okay. @ecstatic-morse could you reduce this PR to just the test case, and remove the intrinsic implementation? We can then merge that once the Rust PR lands. |
type_name.type_name intrinsic.
|
@RalfJung Done! |
|
Could you also remove the implementation of the intrinsic from And please rebase onto current master. |
|
Oh actually never mind, rust-lang/rust#61498 did not land yet. |
|
Tests are failing though: |
|
They will keep failing without a bump of the rustc version we're using in CI |
type_name intrinsic.type_name intrinsic.
|
I'll retrigger a CI run when rust-lang/rust#61498 lands. |
|
You'll also have to update the |
|
I updated |
We bump `rust-version` to pick up the new impl from rust-lang/rust#61498 and add a test.
Addresses this comment from @RalfJung.This makes thetype_nameintrinsic calllibrustc_mir::interpret::type_nameinstead ofTy::to_string.I used the implementation ofeval_const_to_opas a reference.This now adds a test for the
type_nameintrinsic and removes the miri impl in favor of the in-core one.