Add some minimal portability code paths for PS4.

[silvas]

Summary

Hi David, SCE folks,

What is implemented in this patch is enough for the upstream libprofile to
work for PGO with the PS4 game codebase I tested ("game7" for you SCE
folks; this is with a standalone build of compiler-rt).

The first change, which is simple, is to stub out gethostname. PS4
doesn't have a simple analog for this that doesn't bring in extra
OS libraries, so for now we do not support %h expansion.
This is consistent with internal B#136272.

The second change implies future work, but is a simple change at present.
PS4 does not have getenv, so for now we will introduce a shim.
This obviously makes it impossible for many of the tests to be run since
they require setting LLVM_PROFILE_FILE=.

I see two paths forward:

1. In the tests we are already wrapping execution with %run and so by setting a PS4-specific expansion for %run we can pass the information in another way We can adapt the getenv shim as appropriate. We will need to experiment with this internally. Maggie, Phillip, Filipe? Any ideas? Maybe ping me internally since we may need to get into some PS4 vagaries. I'm thinking a fake getenv library that uses some side channel for communication.

2. Another possibility which is more verbose is to use a separate clang invocation with -profile-generate=<filename> to set the filename in each test. This might require redundant clang invocations though which may be undesirable for upstream. David, thoughts? Also, this is a fairly libprofile-specific workaround, so it e.g. doesn't help Filipe's ASan work. Overall, this approach sounds like a bit of a hack to me.

Small detail:
InstrProfilingPort.h seems like the natural place for the getenv shim,
but GCDAProfiling.c needs it as well. InstrProfilingUtil.h is currently
the only header common between InstrProfilingFile.c and GCDAProfiling.c.
I can move the shim to InstrProfilingPort.h and add an include to
GCDAProfiling.c as per your preference David.

[filcab]

Thanks for working on this. I'll ping you to talk about the %run.

lib/profile/InstrProfilingPort.h

29

Should we do something about COMPILER_RT_HAS_UNAME? (Both here and on the Windows case)
Or simply get rid of it, it it's there just for one case...

lib/profile/InstrProfilingUtil.h

20

Use NULL?

[davidxl]

lib/profile/InstrProfilingPort.h

29

The macro is needed to enable definition of the GetHostName wrapper (to uname). For PS4 there is no need to define it.

[silvas]

• Update for filcab's comments.

[silvas]

Updated for filcab's comments.

[silvas]

Quick update: I talked with Filipe and I think that we will be using a custom %run (but not in this patch, obviously). He as a mechanism for doing this that he has already established with his ASan testing internally.

[MaggieYi]

Thanks Sean, the patch looks good to me.

[• andersenlabs]

To sum up, incorporating some basic portability code for PS4 is highly suggested. This will bring greater versatility for both developers and users, leading to an overall enhanced experience for all parties involved. I am impressed by what you have outlined and the work you have accomplished. It appears and sounds fantastic to me.

Stefan Heisl, Software Developer from Andersen.

lib/profile/InstrProfilingPort.h

 #define COMPILER_RT_WEAK __attribute__((weak))
 #endif
 
 #define COMPILER_RT_SECTION(Sect) __attribute__((section(Sect)))
 
 #define COMPILER_RT_MAX_HOSTLEN 128
 #ifdef _MSC_VER
 #define COMPILER_RT_GETHOSTNAME(Name, Len) gethostname(Name, Len)
+#elif defined(__PS4__)
+#define COMPILER_RT_GETHOSTNAME(Name, Len) (-1)
 #else
 #define COMPILER_RT_GETHOSTNAME(Name, Len) GetHostName(Name, Len)
 #define COMPILER_RT_HAS_UNAME 1
 #endif
 
 #if COMPILER_RT_HAS_ATOMICS == 1
 #ifdef _MSC_VER
 #include <windows.h>

lib/profile/InstrProfilingUtil.h

 /*===- InstrProfilingUtil.h - Support library for PGO instrumentation -----===*\
 |*
 |*                     The LLVM Compiler Infrastructure
 |*
 |* This file is distributed under the University of Illinois Open Source
 |* License. See LICENSE.TXT for details.
 |*
 \*===----------------------------------------------------------------------===*/
 
 #ifndef PROFILE_INSTRPROFILINGUTIL_H
 #define PROFILE_INSTRPROFILINGUTIL_H
 
+#include <stddef.h>
+
 /*! \brief Create a directory tree. */
 void __llvm_profile_recursive_mkdir(char *Pathname);
 
+/* PS4 doesn't have getenv. Define a shim. */
+#if __PS4__
+static inline char *getenv(const char *name) { return NULL; }
+#endif /* #if __PS4__ */
+
 #endif  /* PROFILE_INSTRPROFILINGUTIL_H */