Skip to content

Increase iterations for function inlining to 3 once extraction uses less memory #494

Description

@kirstenmg

Currently, extraction uses a lot of memory. Function inlining makes extraction slower. Running cargo run -r tests/passing/small/fib_recursive.bril --run-mode compile-brilift --optimize-egglog true --optimize-brilift true --interp true on its own, for example, takes 7 seconds with 3 iterations of inlining. When run with all other tests (default 8 threads), too much memory is used and this test takes over a minute.

Function inlining is at 2 iterations currently so tests pass in a reasonable amount of time. Once extraction is less memory-intensive, function inlining should be bumped to 3 iterations (in config.rs).

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions