[PATCH] D31831: posix_fallocate isn't support on all filesystems

[Sid Manning via Phabricator via llvm-commits]

sidneym updated this revision to Diff 97457.
sidneym added a comment.

Removed duplicated code.


Repository:
  rL LLVM

https://reviews.llvm.org/D31831

Files:
  lib/Support/Unix/Path.inc


Index: lib/Support/Unix/Path.inc
===================================================================
--- lib/Support/Unix/Path.inc
+++ lib/Support/Unix/Path.inc
@@ -422,13 +422,13 @@
   // If we have posix_fallocate use it. Unlike ftruncate it always allocates
   // space, so we get an error if the disk is full.
   if (int Err = ::posix_fallocate(FD, 0, Size))
-    return std::error_code(Err, std::generic_category());
-#else
+    if (Err != EOPNOTSUPP)
+      return std::error_code(Err, std::generic_category());
+#endif
   // Use ftruncate as a fallback. It may or may not allocate space. At least on
   // OS X with HFS+ it does.
   if (::ftruncate(FD, Size) == -1)
     return std::error_code(errno, std::generic_category());
-#endif
 
   return std::error_code();
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31831.97457.patch
Type: text/x-patch
Size: 776 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170502/5d295416/attachment.bin>