[PATCH] tsan: ignore interceptors coming from specified libraries

Alexander Potapenko glider at google.com
Thu Oct 3 06:22:28 PDT 2013


  LGTM with a nit


================
Comment at: sanitizer_common/sanitizer_libignore.h:66
@@ +65,3 @@
+
+  LibIgnore(const LibIgnore&);  // not implemented
+  void operator = (const LibIgnore&);  // not implemented
----------------
Dmitry Vyukov wrote:
> Alexander Potapenko wrote:
> > I suggest to remove the declarations of non-implemented functions.
> Then it will be possible to copy objects of this class, something that I want to prevent.
Got it now. I just think "not implemented" is a poor comment.
How about "disallow copying LibIgnore objects"?


http://llvm-reviews.chandlerc.com/D1808



More information about the llvm-commits mailing list