[PATCH] D36827: Changed createTemporaryFile without FD to actually create a file.

Ilya Biryukov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 14 07:38:34 PST 2017


ilya-biryukov added a comment.

In https://reviews.llvm.org/D36827#954038, @JDevlieghere wrote:

> Rafael made some changes to have dsymutil use the new TempFile class in r318534 so I think it's no longer relevant there. Is this used anywhere else?


I haven't looked into that yet. Will let you know when I get to it. There was one more usage in `ToolingTest.cpp`, https://reviews.llvm.org/D36828 fixes it.
Good to know that dsymutil does not require any changes.


https://reviews.llvm.org/D36827





More information about the llvm-commits mailing list