8000 Fix fewest modules OOM by steinybot · Pull Request #4986 · scala-js/scala-js · GitHub
[go: up one dir, main page]
Skip to content

Fix fewest modules OOM #4986

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Fix fewest modules OOM #4986

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
88e77d4
Fix #4985: Create separate tagger for FewestModules
steinybot May 20, 2024
7fbe85e
Fix #4985: Tag FewestModules breadth first
steinybot May 20, 2024
3078fac
Fix #4985: Fix transitive static dependencies
steinybot May 20, 2024
310469b
Fix #4985: Fix docs
steinybot May 20, 2024
1644a97
Fix #4985: Combine taggers again
steinybot May 21, 2024
c134cf8
Fix #4985: Update docs
steinybot May 21, 2024
a86e07e
Fix #4985: Update docs
steinybot May 21, 2024
96036ad
Fix #4985: Remove bogus tests
steinybot May 21, 2024
4fdba31
Skip calculating excluded hops for fewest modules
steinybot May 22, 2024
bdfe51f
Add missing exclusion check back in
steinybot May 22, 2024
db851c4
Fix excluded hop counts
steinybot May 22, 2024
b7d9577
Fix revisting static dependencies
steinybot May 22, 2024
f72c81a
Update comment
steinybot May 22, 2024
e47f9d5
Add tests
steinybot May 23, 2024
92d63b5
Update comment
steinybot May 23, 2024
bf0d159
Remove trailing commas
steinybot Jun 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix #4985: Combine taggers again
  • Loading branch information
@steinybot
steinybot committed May 21, 2024
commit 1644a974e15d7d44ff3c2862a901d6aaf4086a9d
2 changes: 1 addition & 1 deletion ...red/src/main/scala/org/scalajs/linker/frontend/modulesplitter/FewestModulesAnalyzer.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ private[modulesplitter] final class FewestModulesAnalyzer extends ModuleAnalyzer
new SingleModuleAnalysis(info.publicModuleDependencies.head._1)
} else {
val modulesToAvoid = info.publicModuleDependencies.keys
val moduleMap = new FewestModulesTagger(info).tagAll(modulesToAvoid)
val moduleMap = new Tagger(info).tagAll(modulesToAvoid)

new FullAnalysis(moduleMap)
}
Expand Down
237 changes: 0 additions & 237 deletions ...hared/src/main/scala/org/scalajs/linker/frontend/modulesplitter/FewestModulesTagger.scala
View file
Open in desktop

This file was deleted.

2 changes: 1 addition & 1 deletion ...d/src/main/scala/org/scalajs/linker/frontend/modulesplitter/SmallModulesForAnalyzer.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ private final class SmallModulesForAnalyzer(

val modulesToAvoid = info.publicModuleDependencies.keys ++ reprToModuleID.values
val largeModuleMap =
new SmallestModulesForTagger(info, excludedClasses = targetClassToRepr.keySet).tagAll(modulesToAvoid)
new Tagger(info, excludedClasses = targetClassToRepr.keySet).tagAll(modulesToAvoid)

new SmallModulesForAnalyzer.Analysis(targetClassToRepr, reprToModuleID, largeModuleMap)
}
Expand Down
Loading
0