abhina.sreeskantharajan marked an inline comment as done.
abhina.sreeskantharajan added inline comments.


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


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99182

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to