[PATCH] D89309: [ThinLTO] In documentation, mention possible values for concurrency flags

[Hans Wennborg via Phabricator via cfe-commits]

hans added a comment.

Thanks for documenting these!

I wonder if including the "CPU sockets" part is more confusing than helpful (especially if the reader doesn't know about CPU sockets). The sockets don't really matter here anyway.

Sometimes the term "physical core" is used in relation to hyper-threading and I find that pretty understandable. How about something line:

- 0: Use one thread per physical core (default)
- 1: Use a single thread only (disable multi-threading)
- all: Use one thread per logical core (uses all hyper threads)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89309/new/

https://reviews.llvm.org/D89309