Uh oh!
There was an error while loading. Please reload this page.
Remove promoted intrinsics caller type checking from const context - #61493
Remove promoted intrinsics caller type checking from const context#61493vertexclique wants to merge 1 commit into
Conversation
rust-highfive
commented
Jun 3, 2019
r? @pnkfelix (rust_highfive has picked a reviewer for you, use r? to override) |
vertexclique
commented
Jun 3, 2019
r? @oli-obk |
RalfJung
commented
Jun 3, 2019
So this does not fix the bug that we allow calling intrinsics in |
Removed part will be added to rust/src/librustc_mir/transform/qualify_consts.rs Lines 1228 to 1230 in 7096ff0 visit_terminator_kind. That will fix the bug seems like. |
| => return true, | ||
| _ => {} | ||
| } |
There was a problem hiding this comment.
@oli-obk We still need a PR that fixes this code and adds a test.
Dylan-DPC-zz
commented
Jul 1, 2019
ping from triage @eddyb@vertexclique@oli-obk any updates? |
vertexclique
commented
Jul 2, 2019
This can be closed, we are not going to use this. We have different PR to solve this one. |
@vertexclique which PR is that? Is it #61835 ? |
vertexclique
commented
Jul 2, 2019
Yes, it's that. It will be here: #61835 |
Removes promoting intrinsics that checks the caller type in
const context. PR linked to issue defined in #61399 (comment)