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

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 11 16:29:51 PST 2020


dexonsmith added a comment.

In D84050#2389985 <https://reviews.llvm.org/D84050#2389985>, @scott.linder wrote:

> As a note, the tests will only deterministically fail with ASan enabled, but I don't know a reliable way to avoid that.

That's fine, we'll always have ASan bots to catch these. Not ideal, but reasonable for this kind of situation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84050



More information about the llvm-commits mailing list