[PATCH] D19167: [esan] EfficiencySanitizer instrumentation pass

[Mike Aizatsky via llvm-commits]

aizatsky added inline comments.

================
Comment at: include/llvm/Transforms/Instrumentation.h:128
@@ +127,3 @@
+
+// Insert SanitizerCoverage instrumentation.
+FunctionPass *createEfficiencySanitizerPass(
----------------
Update comment.

================
Comment at: lib/Transforms/Instrumentation/EfficiencySanitizer.cpp:317
@@ +316,3 @@
+  assert(OrigTy->isSized());
+  uint32_t TypeSize = DL.getTypeStoreSizeInBits(OrigTy);
+  if (TypeSize != 8 && TypeSize != 16 && TypeSize != 32 && TypeSize != 64 &&
----------------
for simplicity/consistency let's make TypeSize in bytes?


http://reviews.llvm.org/D19167