[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