Skip to content

bpo-44249 Update README.rst #26385

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.

Sign up for GitHub

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 10 commits into from
May 28, 2021
Merged

bpo-44249 Update README.rst #26385

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
cc2fa7e
Update README.rst
Ayushparikh-code May 26, 2021
8642a86
📜🤖 Added by blurb_it.
blurb-it[bot] May 26, 2021
d25a681
Update README.rst
Ayushparikh-code May 28, 2021
79d0deb
Update 2021-05-26-18-29-48.bpo-39798.D4UZnR.rst
Ayushparikh-code May 28, 2021
08399cb
Update 2021-05-26-18-29-48.bpo-39798.D4UZnR.rst
terryjreedy May 28, 2021
8911bb6
Update 2021-05-26-18-29-48.bpo-39798.D4UZnR.rst
terryjreedy May 28, 2021
efc7bd4
Update 2021-05-26-18-29-48.bpo-39798.D4UZnR.rst
terryjreedy May 28, 2021
a74a670
Delete 2021-05-26-18-29-48.bpo-39798.D4UZnR.rst
terryjreedy May 28, 2021
35ace7f
Update README.rst
terryjreedy May 28, 2021
b503274
Update README.rst
terryjreedy May 28, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions README.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,12 +110,12 @@ The entire Python directory is cleaned of temporary files that may have
resulted from a previous compilation.

An instrumented version of the interpreter is built, using suitable compiler
flags for each flavour. Note that this is just an intermediary step. The
binary resulting from this step is not good for real life workloads as it has
flags for each flavor. Note that this is just an intermediary step. The
binary resulting from this step is not good for real-life workloads as it has
Comment on lines +113 to +114
Copy link
Member

@terryjreedy terryjreedy May 28, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

These two changes and one other are correct. Will merge when CI done.

profiling instructions embedded inside.

After the instrumented interpreter is built, the Makefile will run a training
workload. This is necessary in order to profile the interpreter execution.
workload. This is necessary in order to profile the interpreter's execution.
Note also that any output, both stdout and stderr, that may appear at this step
is suppressed.

Expand Down