[PATCH] D12869: [Support] Add a new environment variable: LLVM_TEMP_DIR

Steven Wu via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 18:17:28 PDT 2015


steven_wu added a comment.

Can you just use one of the TEMP environmental variable instead? I don't see any difference between your "LLVM_TEMP_DIR" and the temp dir environmental variable defined for Unix or Windows. That is:

  // For Unix
  const char *EnvironmentVariables[] = {"TMPDIR", "TMP", "TEMP", "TEMPDIR"};
  // For Windows
  const char *EnvironmentVariables[] = {"TMP", "TEMP", "USERPROFILE"};          


http://reviews.llvm.org/D12869





More information about the llvm-commits mailing list