[PATCH] D78022: Introduce llvm::sys::Process::getProcessId() and adopt it

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 14:55:22 PDT 2020


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm

Looks like it saves a bunch of ifdefs in otherwise platform neutral code, thanks.



================
Comment at: llvm/include/llvm/Support/Process.h:41
 
 /// A collection of legacy interfaces for querying information about the
 /// current executing process.
----------------
The "legacy interfaces" comment is concerning, but this still seems logical.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78022





More information about the llvm-commits mailing list