Skip to content

Commit c9818eb

Browse files
committedAug 17, 2018
[ThinLTO] Correct documentation on default number of threads
Summary: The number of threads used for ThinLTO backend parallelism was dropped to the number of cores in r284618 to avoid oversubscribing physical cores due to hyperthreading. This updates the documentation to reflect that change. Fixes PR38610. Reviewers: pcc Subscribers: mehdi_amini, inglorion, eraman, steven_wu, dexonsmith, cfe-commits Differential Revision: https://reviews.llvm.org/D50882 llvm-svn: 340021
1 parent 16a2f54 commit c9818eb

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed
 

‎clang/docs/ThinLTO.rst

+3-1
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,9 @@ Controlling Backend Parallelism
105105
-------------------------------
106106
.. _parallelism:
107107

108-
By default, the ThinLTO link step will launch up to
108+
By default, the ThinLTO link step will launch as many
109+
threads in parallel as there are cores. If the number of
110+
cores can't be computed for the architecture, then it will launch
109111
``std::thread::hardware_concurrency`` number of threads in parallel.
110112
For machines with hyper-threading, this is the total number of
111113
virtual cores. For some applications and machine configurations this

0 commit comments

Comments
 (0)
Please sign in to comment.