]> git.lizzy.rs Git - rust.git/commit
Auto merge of #106340 - saethlin:propagate-operands, r=oli-obk
authorbors <bors@rust-lang.org>
Mon, 9 Jan 2023 11:59:51 +0000 (11:59 +0000)
committerbors <bors@rust-lang.org>
Mon, 9 Jan 2023 11:59:51 +0000 (11:59 +0000)
commit89e0576bd3aec2f1acf935b353900bd75742860b
tree832426029cb2ca75232215aa44aba6a40a479954
parentc54c8cbac882e149e04a9e1f2d146fd548ae30ae
parent82f0973dd52d113666304802e9213b089e3211c6
Auto merge of #106340 - saethlin:propagate-operands, r=oli-obk

Always permit ConstProp to exploit arithmetic identities

Fixes https://github.com/rust-lang/rust/issues/72751

Initially, I thought I would need to enable operand propagation then do something else, but actually https://github.com/rust-lang/rust/pull/74491 already has the fix for the issue in question! It looks like this optimization was put under MIR opt level 3 due to possible soundness/stability implications, then demoted further to MIR opt level 4 when MIR opt level 2 became associated with `--release`.

Perhaps in the past we were doing CTFE on optimized MIR? We aren't anymore, so this optimization has no stability implications.

r? `@oli-obk`