-
Notifications
You must be signed in to change notification settings - Fork 24
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.
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
perf(compiler): Inliner optimization [LNG-322] #1047
Merged
Merged
Changes from 15 commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
5269fe9
String -> StringBuilder, debug stuff
DieMyst 0baab9c
Merge remote-tracking branch 'origin/main' into inliner-optimization
DieMyst 711e58f
bump
DieMyst f3137f4
optimize Mangler
DieMyst 83f994b
Merge remote-tracking branch 'origin/main' into inliner-optimization
DieMyst e34c155
fix compilation
DieMyst e925d3d
fix tests
DieMyst a64ad70
fix tests
DieMyst a902446
revert
DieMyst 8a5b9af
refactoring algorithm, tests
DieMyst c960320
fix algorithm
DieMyst d6986a1
clear PR
DieMyst 6183051
Merge remote-tracking branch 'origin/main' into inliner-optimization
DieMyst 40c4efc
add comments
DieMyst c92755b
refactor
DieMyst b5c66d0
PR fixes
DieMyst d1eb309
Merge branch 'main' into inliner-optimization
DieMyst File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,40 +1,33 @@ | ||
aqua M | ||
|
||
export bugLng317 | ||
export returnSrvAsAbility | ||
|
||
service MyOp("op"): | ||
identity(s: string) -> string | ||
ability MyAb: | ||
call() -> string | ||
|
||
ability WorkerJob: | ||
runOnSingleWorker(w: string) -> []string | ||
service MySrv("default-id"): | ||
call() -> string | ||
|
||
func runJob(j: -> []string) -> []string: | ||
<- j() | ||
func mySrvDefault() -> MyAb: | ||
<- MySrv | ||
|
||
func disjoint_run{WorkerJob}() -> -> []string: | ||
run = func () -> []string: | ||
r <- WorkerJob.runOnSingleWorker("a") | ||
<- r | ||
<- run | ||
func mySrvResolved() -> MyAb: | ||
MySrv "resolved-id" | ||
<- MySrv | ||
|
||
func empty() -> string: | ||
a = "empty" | ||
<- a | ||
func mySrvThird() -> MyAb: | ||
MySrv "third-id" | ||
<- MySrv | ||
|
||
func bugLng317() -> []string: | ||
func useMyAb{MyAb}() -> string: | ||
<- MyAb.call() | ||
|
||
res: *string | ||
|
||
outer = () -> string: | ||
<- empty() | ||
|
||
clos = () -> -> []string: | ||
job2 = () -> []string: | ||
res <- outer() | ||
res <- MyOp.identity("identity") | ||
<- res | ||
<- job2 | ||
worker_job = WorkerJob(runOnSingleWorker = clos()) | ||
subnet_job <- disjoint_run{worker_job}() | ||
finalRes <- runJob(subnet_job) | ||
<- finalRes | ||
func returnSrvAsAbility() -> []string: | ||
result: *string | ||
MySrvDefault <- mySrvDefault() | ||
MySrvResolved <- mySrvResolved() | ||
MySrvThird <- mySrvThird() | ||
result <- useMyAb{MySrvDefault}() | ||
result <- useMyAb{MySrvResolved}() | ||
result <- useMyAb{MySrvThird}() | ||
<- result |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,7 +11,7 @@ class ImportsSpec extends AnyFlatSpec with ScalaCheckPropertyChecks with Matcher | |
|
||
implicit override val generatorDrivenConfig = | ||
// Tests here are lightweight, so we can afford to run more of them | ||
PropertyCheckConfiguration(minSuccessful = 500, sizeRange = 64) | ||
PropertyCheckConfiguration(minSuccessful = 50, sizeRange = 32) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why lower those values? |
||
|
||
val shortAlphaNumStr = for { | ||
length <- Gen.choose(1, 10) | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you restore this file please?