[PATCH] D24811: [sanitizer_common] Delete some copy/move methods in InternalScopedBuffer

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 21 11:24:47 PDT 2016


kcc accepted this revision.
kcc added a comment.
This revision is now accepted and ready to land.

LGTM

When sending a patch for review, try to not mix style changes and code changes.

> > code that takes ownership of an InternalScopedString

> 


I am not sure I want any kind of such complexity in this code, but let's see the patches.

(Also, please stop CC-ing samsonov in code reviews here, he is no longer working on LLVM)


https://reviews.llvm.org/D24811





More information about the llvm-commits mailing list