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

Daniel Waters via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 15:01:58 PST 2016


grotdunst added inline comments.

================
Comment at: lib/profile/InstrProfilingFile.c:159
@@ +158,3 @@
+          HostNameLength = strlen(Hostname);
+          if (HostNameLength <= 0)
+            return -1;
----------------
davidxl wrote:
> Is this check necessary?
You're right: it's unnecessary because of getHostName. I'll remove it.


http://reviews.llvm.org/D16371





More information about the llvm-commits mailing list