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

Document --profiling-detail in setup-commands. #9388

Merged
merged 1 commit into from
Nov 11, 2023

Conversation

AndreasPK
Copy link
Collaborator

#9182 pointed out that there is some missing documentation.

Copy link
Member

@Mikolaj Mikolaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you.

late-toplevel
Like top-level but costs will be assigned to top level definitions after
optimization. This lowers profiling overhead massively while giving similar
levels of detail as toplevle-functions. However it means functions introduced
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
levels of detail as toplevle-functions. However it means functions introduced
levels of detail as toplevel-functions. However it means functions introduced

@Kleidukos Kleidukos added merge me Tell Mergify Bot to merge and removed attention: needs-review labels Nov 9, 2023
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Nov 11, 2023
@Mikolaj Mikolaj force-pushed the wip/andreask/late-docs branch from dd7e58b to ef0ca48 Compare November 11, 2023 11:56
@mergify mergify bot merged commit 69417ac into haskell:master Nov 11, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days merge me Tell Mergify Bot to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants