[Openmp-commits] [PATCH] D13175: Added sockets to the syntax of KMP_PLACE_THREADS environment variable.

[Andrey Churbanov via Openmp-commits]

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

LGTM

Jim,

We can think of your suggestion as a separate code improvement, given that we make syntax of the KMP_PLACE_THREADS stricter (mandatory delimiters, strict order of elements).

Currently delimiters are optional, and some freedom in order of elements allowed to support initial socket-less syntax. That makes the structured loopy parsing problematic.


Repository:
  rL LLVM

http://reviews.llvm.org/D13175