]> git.lizzy.rs Git - rust.git/commit - compiler/rustc_codegen_cranelift/src/driver/mod.rs
Rollup merge of #105683 - JakobDegen:dest-prop-storage, r=tmiasko
authorMatthias Krüger <matthias.krueger@famsik.de>
Thu, 15 Dec 2022 11:46:02 +0000 (12:46 +0100)
committerGitHub <noreply@github.com>
Thu, 15 Dec 2022 11:46:02 +0000 (12:46 +0100)
commit6cdc83b64e660356f7171b96125304ae91e1c29b
tree0a1a8b40e8b84fbe260f4977d000091d82a6386e
parent78cf8cc02e26b20e8d76a7a2557defd6072dda1c
parenta5beb7abb9da221f73e839f555835c36716e3015
Rollup merge of #105683 - JakobDegen:dest-prop-storage, r=tmiasko

Various cleanups to dest prop

This makes fixing the issues identified in #105577 easier. A couple changes

 - Use an enum with names instead of a bool
 - Only call `remove_candidates_if` from one place instead of two. Doing it from two places is far too fragile, since any divergence in the behavior between those callsites is likely to be unsound.
 - Remove `is_constant`. Right now we only merge locals, so this doesn't do anything, and the logic would be wrong if it did.

r? `@tmiasko`