[PATCH] D16371: [compiler-rt/profile] Added Hostname to .profdata file

[Hal Finkel via llvm-commits]

hfinkel added a comment.

In http://reviews.llvm.org/D16371#338901, @vsk wrote:

> To clarify, my objection to using guards for this feature wasn't cosmetic: is there anything fundamental about MSVC that inhibits this feature?


There's no uname() (or sys/utsname.h) on Windows. That's a POSIX function.

> I see that your lit test tests this patch, but (1) am concerned that the usage of uname might break Windows bots, and


It would, except that the '// REQUIRES: shell' should cause the Windows bots to skip the test.

> (2) wonder whether uname can be used by your scripts directly instead of relying on %h-expansion.


It is not clear how else to test that the host-name expansion actually works.


http://reviews.llvm.org/D16371