Skip to content

Make it clearer how the user can control compiler optimization #530

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

Merged
merged 7 commits into from
Sep 21, 2021

Conversation

gareth-nx
Copy link
Contributor

These edits to the README attempt to offer simple guidance on compiling stdlib with optimization flags.

In this discussion here it became clear that not everyone understands how to apply compiler optimizations to stdlib, although the discussion shows that is quite important for performance.

I think some simple pointers would help users who do not have much experience with build tools (like me!).

@awvwgk awvwgk added build: cmake Issue with stdlib's CMake build files build: make Issue with stdlib's manual makefile build documentation Improvements or additions to documentation reviewers needed This patch requires extra eyes labels Sep 19, 2021
Copy link
Member

@awvwgk awvwgk left a comment

Choose a reason for hiding this comment

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

Thanks for sharing, looks good to me.

Copy link
Member

@jvdp1 jvdp1 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 for these additions. I approve pending minor comments.

gareth-nx and others added 2 commits September 21, 2021 08:53
Co-authored-by: Jeremie Vandenplas <jeremie.vandenplas@gmail.com>
Co-authored-by: Jeremie Vandenplas <jeremie.vandenplas@gmail.com>
Copy link
Member

@milancurcic milancurcic 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, @gareth-nx!

@milancurcic milancurcic merged commit 5614737 into fortran-lang:master Sep 21, 2021
@awvwgk awvwgk removed the reviewers needed This patch requires extra eyes label Sep 25, 2021
@gareth-nx gareth-nx deleted the pr/readme_updates branch October 7, 2021 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build: cmake Issue with stdlib's CMake build files build: make Issue with stdlib's manual makefile build documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants