[PATCH] D84050: YAML: Don't assume an arbitrary StringRef is null terminated

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D84050#2189370 <https://reviews.llvm.org/D84050#2189370>, @bkramer wrote:

> The parser used to rely on the `\0` being present. Was that fixed?

Do you know where this assumption was? I couldn't find it, but also couldn't find any commits that seemed to "fix" it. I haven't seen a sanitizer error yet


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

https://reviews.llvm.org/D84050