T0b1/regalloc2
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Always recompute prio when recomputing bundle properties; otherwise is zero on new bundles and causes spill weight to be zeroed

Browse Source
This commit is contained in:
Chris Fallin
2021-06-11 13:21:50 -07:00
parent ea814225a2
commit a686d5a513
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/ion/mod.rs
View File

@@ -2823,6 +2823,8 @@ impl<'a, F: Function> Env<'a, F> {
let first_range = bundledata.ranges[0].index; let first_range = bundledata.ranges[0].index;
let first_range_data = &self.ranges[first_range.index()]; let first_range_data = &self.ranges[first_range.index()];
self.bundles[bundle.index()].prio = self.compute_bundle_prio(bundle);
if first_range_data.vreg.is_invalid() { if first_range_data.vreg.is_invalid() {
log::debug!(" -> no vreg; minimal and fixed"); log::debug!(" -> no vreg; minimal and fixed");
minimal = true; minimal = true;
@@ -3264,17 +3266,15 @@ impl<'a, F: Function> Env<'a, F> {
if self.bundles[bundle.index()].ranges.len() > 0 { if self.bundles[bundle.index()].ranges.len() > 0 {
self.recompute_bundle_properties(bundle); self.recompute_bundle_properties(bundle);
let prio = self.compute_bundle_prio(bundle); let prio = self.bundles[bundle.index()].prio;
self.bundles[bundle.index()].prio = prio;
self.allocation_queue self.allocation_queue
.insert(bundle, prio as usize, reg_hint); .insert(bundle, prio as usize, reg_hint);
} }
if self.bundles[new_bundle.index()].ranges.len() > 0 { if self.bundles[new_bundle.index()].ranges.len() > 0 {
self.recompute_bundle_properties(new_bundle); self.recompute_bundle_properties(new_bundle);
let new_prio = self.compute_bundle_prio(new_bundle); let prio = self.bundles[new_bundle.index()].prio;
self.bundles[new_bundle.index()].prio = new_prio;
self.allocation_queue self.allocation_queue
.insert(new_bundle, new_prio as usize, reg_hint); .insert(new_bundle, prio as usize, reg_hint);
} }
} }