[PATCH] D23672: Start reifying error descriptions

Filipe Cabecinhas via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 26 10:28:06 PDT 2016


filcab added inline comments.

================
Comment at: lib/asan/asan_errors.h:50
@@ +49,3 @@
+  ErrorKind kind;
+  // (filcab): I'd rather avoid inheritance + virtual functions since that makes
+  // the type non-trivially copiable. I would like to have the same structures
----------------
kcc wrote:
> Sorry, I did not mean to literally copy the discussion into the comments.
> The comment should not be in a form of discussion, but in a form of statement. 
> 
> // We do X because Y and Z
Wow, that was especially lazy. I've rewritten the comment now. I think it contains the main points.


https://reviews.llvm.org/D23672





More information about the llvm-commits mailing list