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

[Bob Haarman via Phabricator via llvm-commits]

inglorion added a comment.

Thanks for doing this! I did some of the ground work for this in the COFF linker, but never got around to flipping the default. I too like the idea of having a separate function for the NUL-terminated case. It seems we all want that, so how do we get there? Are you planning to follow this up with a patch that removes the RequiresNullTerminator flag and rewrites calls that use it to use the new function instead?


https://reviews.llvm.org/D46205