[PATCH] D19167: [esan] EfficiencySanitizer instrumentation pass

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 16:55:41 PDT 2016


pcc added a subscriber: pcc.

================
Comment at: lib/Transforms/Instrumentation/EfficiencySanitizer.cpp:21
@@ +20,3 @@
+
+#include "llvm/Transforms/Instrumentation.h"
+#include "llvm/ADT/SmallString.h"
----------------
bruening wrote:
> aizatsky wrote:
> > wrong import order. Please run clang-tidy & clang-format on the file.
> Will do.  Note that AddressSanitizer.cpp, MemorySanitizer, ThreadSanitizer.cpp, etc. on which this file is based all have this same (incorrect) order.
No, see http://llvm.org/docs/CodingStandards.html#a-public-header-file-is-a-module third paragraph.


http://reviews.llvm.org/D19167





More information about the llvm-commits mailing list