[PATCH] D75153: [ThinLTO] Allow usage of all SMT threads in the system

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 1 09:51:08 PST 2020


mehdi_amini added a comment.

> /opt:lldltojobs=N -- limit usage to N threads, but constrained by usage of heavyweight_hardware_concurrency().

I really dislike this behavior: this seems user hostile to me. I would either:

- honor the user request (eventually display a warning), this is in line with other system behavior like `ninja -j N` for instance.
- reject the user request

If you want such a behavior, then it should be another flag which express it in the name like `opt:lldltomaxnumjobs`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75153





More information about the llvm-commits mailing list