[PATCH] D99182: [NFC] Reordering parameters in getFile and getFileOrSTDIN

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 25 01:42:49 PDT 2021


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

LGTM, with nit.



================
Comment at: llvm/lib/Support/MemoryBuffer.cpp:275
+  return getFileAux<WritableMemoryBuffer>(
+      Filename, /*MapSize=*/-1, 0, /*IsText=*/false,
+      /*RequiresNullTerminator=*/false, IsVolatile);
----------------
Whilst you're modifying, add the named parameter comment for the `0`, so that it's not just a magic number.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99182



More information about the llvm-commits mailing list