search

joverbey / oaccrefac

1 stars 1 forks source link

Fusion dependence #49

Closed aggrand closed 8 years ago

acalvert commented 8 years ago

I've added a couple of fusion tests since this PR was made. The code all looks good to me if it passes the UI tests. I'm glad making a fake parent node ended up working; seems like the cleanest way to do it.

acalvert commented 8 years ago

Also worth noting: The AbstractDependenceAnalysis only existed in the first place to share code between the regular and fusion analyses. Since the fusion analysis no longer exists, all of AbstractDependenceAnalysis could probably be moved back down into the regular dependence analysis so that the abstract one can be removed altogether as well.