Skip to content
Snippets Groups Projects
Commit 70bb396d authored by rarbore2's avatar rarbore2
Browse files

Merge branch 'ignore_antideps' into 'main'

Ignore anti-deps in GCM for now

See merge request !87
parents 19c7dde3 d3a245b5
No related branches found
No related tags found
1 merge request!87Ignore anti-deps in GCM for now
Pipeline #200769 passed
......@@ -73,14 +73,15 @@ pub fn gcm(
.as_ref()
.into_iter()
.map(|id| *id)
// TODO: once anti-dependence analysis is fixed, use it in GCM.
// Include "uses" from anti-dependencies.
.chain(
antideps_uses
.get(&id)
.unwrap_or(&vec![])
.into_iter()
.map(|id| *id),
)
//.chain(
// antideps_uses
// .get(&id)
// .unwrap_or(&vec![])
// .into_iter()
// .map(|id| *id),
//)
.map(|id| schedule_early[id.idx()])
.collect();
if let Some(use_places) = use_places {
......@@ -127,19 +128,16 @@ pub fn gcm(
// For every user, consider where we need to be to directly dominate the
// user.
for user in def_use
.get_users(id)
.as_ref()
.into_iter()
.map(|id| *id)
// Include "users" from anti-dependencies.
.chain(
antideps_users
.get(&id)
.unwrap_or(&vec![])
.into_iter()
.map(|id| *id),
)
for user in def_use.get_users(id).as_ref().into_iter().map(|id| *id)
// TODO: once anti-dependence analysis is fixed, use it in GCM.
// Include "users" from anti-dependencies.
//.chain(
// antideps_users
// .get(&id)
// .unwrap_or(&vec![])
// .into_iter()
// .map(|id| *id),
//)
{
if let Node::Phi { control, data } = &function.nodes[user.idx()] {
// For phis, we need to dominate the block jumping to the phi in
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment