[PATCH] D47460: Treat files as volatile by default

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 30 06:12:48 PDT 2018


ilya-biryukov requested changes to this revision.
ilya-biryukov added a comment.
This revision now requires changes to proceed.

It's a very subtle change and it isn't clear if that's the right thing to do without understanding the problem that we're trying to solve.
Could you please elaborate on the problem?

>   accidental memory mapping for main file which sometimes happens through FileSystem::getBufferForFile

Why is memory mapping considered accidental and what problems does it cause?


https://reviews.llvm.org/D47460





More information about the cfe-commits mailing list