[PATCH] D77772: [Clang] Expose RequiresNullTerminator in FileManager.

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 9 15:48:45 PDT 2020


dexonsmith added a comment.

Code change LGTM.

Is there a way to add a `MemoryBuffer` unit test for the change to `shouldUseMmap`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77772





More information about the cfe-commits mailing list