[PATCH] D27295: Remove existing file in a separate thread asynchronously.

Joerg Sonnenberger via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 07:00:25 PST 2016


On Fri, Dec 09, 2016 at 09:46:37AM -0500, Ed Maste via llvm-commits wrote:
> On 7 December 2016 at 17:34, Rafael Avila de Espindola
> <rafael.espindola at gmail.com> wrote:
> > I think the correct way to this is
> >
> > * Don't remove the existing file.
> > * Create a temporary file with no name using O_TMPFILE
> > * Use renameat to give it the correct name at the end of the link.
> 
> O_TMPFILE's not POSIX though so we'd need a fallback path.

Even worse, renameat would rule out a lot of older systems for no good
reason.

Joerg


More information about the llvm-commits mailing list