[PATCH] D59130: [llvm][Support] Provide interface to set thread priorities

Jan Korous via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 18 11:31:13 PDT 2019


jkorous accepted this revision.
jkorous added a comment.

Gotchas and symptoms are IMO dependent on the use-case. Since this is a generic API I'd say that documenting the behavior is the best we can do here.

So how about @kadircet just adds a note to the header that platform specific details might be important here (depending on the use-case) and that more information is in comments in relevant implementation? Is that what you mean @jfb?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59130





More information about the llvm-commits mailing list