[PATCH] D146490: [Support] On Windows, ensure that UniqueID is really stable
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 21 05:44:41 PDT 2023
aaron.ballman added a comment.
FWIW, I'm also nervous about this change. Keeping the file handle open changes the ways in which you can access the file from other processes (or within the same process), so I'd expect this to cause some heartache, possibly for downstreams as well as Clang users. I mostly worry about things like leaving a file open interacting with things like deleting the file from the file system, including via a user interaction.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146490/new/
https://reviews.llvm.org/D146490
More information about the cfe-commits
mailing list