Skip to content
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

Bottlenecks in standard library #9

Open
mrhaandi opened this issue Jul 20, 2023 · 1 comment
Open

Bottlenecks in standard library #9

mrhaandi opened this issue Jul 20, 2023 · 1 comment

Comments

@mrhaandi
Copy link
Contributor

Description of the problem

The standard library contains files such as Reals/Ranalysis.v, which mainly contain Require Export xyz.
This is great for users (no need to search the exact file names). However, when used in the standard library internally, this creates a bottleneck with respect to parallel compilation of the standard library.

For example, Reals/NewtonInt.v contains Require Import Ranalysis., which (among others) exports Rgeom.
However, Rgeom is not necessary for NewtonInt. Still in parallel compilation NewtonInt needs Rgeom to compile first.

Avoiding unnecessary bottlenecks by better import management in the standard library may improve parallel compilation speed.

Coq Version

8.17

@herbelin
Copy link
Member

That's a good point. The original design of Reals had this chain of Export but more parallelism would indeed be nice. If you already made such finer analysis, that would be worth a PR (in my personal opinion).

Villetaneuse referenced this issue in Villetaneuse/ceps Sep 29, 2023
Also, mention Math-classes and coq/coq#17877
@proux01 proux01 transferred this issue from coq/coq Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants