[PATCH] D122391: [llvm] Fix string copy confusion

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 24 09:27:52 PDT 2022


dblaikie accepted this revision.
dblaikie added a comment.

Yeah, it wouldn't /entirely/ surprise me (knowing little about the code) if something, somewhere, depended on the StringView actually pointing to a null terminated string - but if tests are passing, etc (especially with something like asan), then this sounds OK to me.


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

https://reviews.llvm.org/D122391



More information about the llvm-commits mailing list