[PATCH] D47960: [asan, myriad] Support environment variables

[Walter Lee via Phabricator via llvm-commits]

waltl updated this revision to Diff 150892.
waltl added a comment.

Sorry for the overly complicated initial patch.  We've decided to get
setenv() to work in our test enviroment (using similar approach to the
initial patch here), so now the sanitizer run-time can just call that.


Repository:
  rL LLVM

https://reviews.llvm.org/D47960

Files:
  compiler-rt/lib/sanitizer_common/sanitizer_rtems.cc


Index: compiler-rt/lib/sanitizer_common/sanitizer_rtems.cc
===================================================================
--- compiler-rt/lib/sanitizer_common/sanitizer_rtems.cc
+++ compiler-rt/lib/sanitizer_common/sanitizer_rtems.cc
@@ -239,8 +239,11 @@
   return true;
 }
 
-char **GetArgv() { return NULL; }
-const char *GetEnv(const char *name) { return NULL; }
+char **GetArgv() { return nullptr; }
+
+const char *GetEnv(const char *name) {
+  return getenv(name);
+}
 
 uptr ReadBinaryName(/*out*/char *buf, uptr buf_len) {
   internal_strncpy(buf, "StubBinaryName", buf_len);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47960.150892.patch
Type: text/x-patch
Size: 588 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180612/251e6a38/attachment.bin>