[PATCH] D91421: Fix temporary file name on Windows

[Artem Belevich via Phabricator via cfe-commits]

tra accepted this revision.
tra added a comment.
This revision is now accepted and ready to land.

LGTM overall, nit about the character choice.



================
Comment at: clang/lib/Driver/Driver.cpp:4645
+  // therefore replace it with '%'.
+  std::replace(BoundArch.begin(), BoundArch.end(), ':', '%');
+#endif
----------------
Windows `cmd` shell uses `%` to reference environment variables. The character is valid for a path, but that would be similar to using `$` in the path name on Linux -- doable, but rather inconvenient to deal with.

Perhaps a less special character would be a better choice. `_`, `.`,  or `@` ?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91421/new/

https://reviews.llvm.org/D91421