[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 12:58:07 PDT 2018
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
More information about the llvm-commits
mailing list