[LLVMdev] sys::path::system_temp_directory vs. sys::fs::createTemporaryFile

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Jul 25 13:44:01 PDT 2014


On 25 July 2014 16:31, David Majnemer <david.majnemer at gmail.com> wrote:
> I have a few concerns:
> 1. GetTempPathW is buggy, we should be manually using
> GetEnvironmentVariableW with TMP, TEMP and USERPROFILE arguments and use the
> first that exists.
> 2. We should make sure that the platform specific pieces live in their
> respective Path.inc files.
>

But that is orthogonal, no?

If I understand correctly, Jordan's question is "do we need both
system_temp_directory and TempDir"? I doesn't look like it, it seems
that every user of TempDir could be replaced with
system_temp_directory.

Cheers,
Rafael



More information about the llvm-dev mailing list