[PATCH] D116603: [AIX][z/OS][Support] Provide no-op mapped_file_region::dontNeedImpl implementation

Abhina Sree via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 5 06:42:55 PST 2022


abhina.sreeskantharajan accepted this revision.
abhina.sreeskantharajan added a comment.
This revision is now accepted and ready to land.

LGTM with minor formatting comment



================
Comment at: llvm/lib/Support/Unix/Path.inc:878
+#else
   if (Mapping)
     ::madvise(Mapping, Size, MADV_DONTNEED);
----------------
Can this be #endif instead of #else? Not sure if this counts as not using an else after return.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116603/new/

https://reviews.llvm.org/D116603



More information about the llvm-commits mailing list