[all-commits] [llvm/llvm-project] fd0d84: [flang][runtime] Support all non-ASCII characters ...

Markus Mützel via All-commits all-commits at lists.llvm.org
Sat May 27 05:54:37 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fd0d846633b05e1d0072754aea7f3985be36c0dc
      https://github.com/llvm/llvm-project/commit/fd0d846633b05e1d0072754aea7f3985be36c0dc
  Author: Markus Mützel <markus.muetzel at gmx.de>
  Date:   2023-05-27 (Sat, 27 May 2023)

  Changed paths:
    M flang/runtime/file.cpp

  Log Message:
  -----------
  [flang][runtime] Support all non-ASCII characters in temporary path on Windows

If the path to the TEMP folder contains (non-ASCII) characters that cannot be
encoded in the current 8-bit locale of the user, openfile_mkstemp might fail
on Windows.
That is an unlikely scenario. But given that the path to the default TEMP
folder on Windows contains the Windows user name, it is still possible.

Use the wide character Windows API to avoid that (unlikely) issue.

Reviewed By: vzakhari

Differential Revision: https://reviews.llvm.org/D151571




More information about the All-commits mailing list