[PATCH] D19842: In openFileForRead, attempt to fetch the actual name of the file on disk -- including case -- so that clang can later warn about non-portable #include and #import directives.

[Eric Niebler via llvm-commits]

eric_niebler added inline comments.

================
Comment at: lib/Support/Unix/Path.inc:553
@@ +552,3 @@
+namespace {
+  bool hasProcSelfFD() {
+    // If we have a /proc filesystem mounted, we can quickly establish the
----------------
bruno wrote:
> Nice, I missed that it was used more than once. Please make hasProcSelfFD() static as well and move it inside llvm::sys::fs.
> Please make hasProcSelfFD() static

It's in an anonymous namespace. What is the reason for preferring `static`?

> and move it inside llvm::sys::fs

It's already there.


http://reviews.llvm.org/D19842