[PATCH] D46205: Set MemoryBuffer's RequiresNullTerminator to false by default.

Bob Haarman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 8 16:26:22 PDT 2018


inglorion added a comment.

I haven't accepted the patch yet because I want to make sure we do end up removing the RequiresNullTerminator flag. You indicated that you don't want to do that in this patch. That's ok with me as long as you do plan to do it in a soon-to-come follow-up patch. If you don't plan to write that follow-up patch, I would like you to include the new function and rewrite the calls in this patch.


https://reviews.llvm.org/D46205





More information about the llvm-commits mailing list