blob: 9331e1d2b3fdd1cb7f1e059ba7db98e7320ede41 [file] [log] [blame]
Kostya Serebryanye2a0e412012-02-13 22:50:51 +00001//===-- ThreadSanitizer.cpp - race detector -------------------------------===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// This file is a part of ThreadSanitizer, a race detector.
11//
12// The tool is under development, for the details about previous versions see
13// http://code.google.com/p/data-race-test
14//
15// The instrumentation phase is quite simple:
16// - Insert calls to run-time library before every memory access.
17// - Optimizations may apply to avoid instrumenting some of the accesses.
18// - Insert calls at function entry/exit.
19// The rest is handled by the run-time library.
20//===----------------------------------------------------------------------===//
21
Chandler Carruthed0881b2012-12-03 16:50:05 +000022#include "llvm/Transforms/Instrumentation.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000023#include "llvm/ADT/SmallSet.h"
24#include "llvm/ADT/SmallString.h"
25#include "llvm/ADT/SmallVector.h"
26#include "llvm/ADT/Statistic.h"
27#include "llvm/ADT/StringExtras.h"
Benjamin Kramer799003b2015-03-23 19:32:43 +000028#include "llvm/Analysis/CaptureTracking.h"
29#include "llvm/Analysis/ValueTracking.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000030#include "llvm/IR/DataLayout.h"
31#include "llvm/IR/Function.h"
32#include "llvm/IR/IRBuilder.h"
Kostya Serebryany463aa812013-03-28 11:21:13 +000033#include "llvm/IR/IntrinsicInst.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000034#include "llvm/IR/Intrinsics.h"
35#include "llvm/IR/LLVMContext.h"
36#include "llvm/IR/Metadata.h"
37#include "llvm/IR/Module.h"
38#include "llvm/IR/Type.h"
Kostya Serebryanyabad0022012-03-14 23:33:24 +000039#include "llvm/Support/CommandLine.h"
Kostya Serebryanye2a0e412012-02-13 22:50:51 +000040#include "llvm/Support/Debug.h"
Kostya Serebryanye2a0e412012-02-13 22:50:51 +000041#include "llvm/Support/MathExtras.h"
Kostya Serebryany6f8a7762012-03-26 17:35:03 +000042#include "llvm/Support/raw_ostream.h"
Kostya Serebryanya1259772012-04-27 07:31:53 +000043#include "llvm/Transforms/Utils/BasicBlockUtils.h"
Kostya Serebryanye2a0e412012-02-13 22:50:51 +000044#include "llvm/Transforms/Utils/ModuleUtils.h"
Kostya Serebryanye2a0e412012-02-13 22:50:51 +000045
46using namespace llvm;
47
Chandler Carruth964daaa2014-04-22 02:55:47 +000048#define DEBUG_TYPE "tsan"
49
Kostya Serebryanyd23b18f2012-10-04 05:28:50 +000050static cl::opt<bool> ClInstrumentMemoryAccesses(
51 "tsan-instrument-memory-accesses", cl::init(true),
52 cl::desc("Instrument memory accesses"), cl::Hidden);
53static cl::opt<bool> ClInstrumentFuncEntryExit(
54 "tsan-instrument-func-entry-exit", cl::init(true),
55 cl::desc("Instrument function entry and exit"), cl::Hidden);
56static cl::opt<bool> ClInstrumentAtomics(
57 "tsan-instrument-atomics", cl::init(true),
58 cl::desc("Instrument atomics"), cl::Hidden);
Kostya Serebryany463aa812013-03-28 11:21:13 +000059static cl::opt<bool> ClInstrumentMemIntrinsics(
60 "tsan-instrument-memintrinsics", cl::init(true),
61 cl::desc("Instrument memintrinsics (memset/memcpy/memmove)"), cl::Hidden);
Kostya Serebryanyabad0022012-03-14 23:33:24 +000062
Kostya Serebryany5a4b7a22012-04-23 08:44:59 +000063STATISTIC(NumInstrumentedReads, "Number of instrumented reads");
64STATISTIC(NumInstrumentedWrites, "Number of instrumented writes");
Alexey Samsonovf54e3aa2012-08-30 13:47:13 +000065STATISTIC(NumOmittedReadsBeforeWrite,
Kostya Serebryany5a4b7a22012-04-23 08:44:59 +000066 "Number of reads ignored due to following writes");
67STATISTIC(NumAccessesWithBadSize, "Number of accesses with bad size");
68STATISTIC(NumInstrumentedVtableWrites, "Number of vtable ptr writes");
Dmitry Vyukov55e63ef2013-03-22 08:51:22 +000069STATISTIC(NumInstrumentedVtableReads, "Number of vtable ptr reads");
Kostya Serebryany5a4b7a22012-04-23 08:44:59 +000070STATISTIC(NumOmittedReadsFromConstantGlobals,
71 "Number of reads from constant globals");
72STATISTIC(NumOmittedReadsFromVtable, "Number of vtable reads");
Dmitry Vyukov2e8d82e2015-02-12 09:55:28 +000073STATISTIC(NumOmittedNonCaptured, "Number of accesses ignored due to capturing");
Kostya Serebryanybf2de802012-04-10 18:18:56 +000074
Ismail Pazarbasi2d4ae9f2015-05-07 21:41:23 +000075static const char *const kTsanModuleCtorName = "tsan.module_ctor";
76static const char *const kTsanInitName = "__tsan_init";
77
Kostya Serebryanye2a0e412012-02-13 22:50:51 +000078namespace {
Kostya Serebryanybf2de802012-04-10 18:18:56 +000079
Kostya Serebryanye2a0e412012-02-13 22:50:51 +000080/// ThreadSanitizer: instrument the code in module to find races.
81struct ThreadSanitizer : public FunctionPass {
Mehdi Aminia28d91d2015-03-10 02:37:25 +000082 ThreadSanitizer() : FunctionPass(ID) {}
Craig Topper3e4c6972014-03-05 09:10:37 +000083 const char *getPassName() const override;
84 bool runOnFunction(Function &F) override;
85 bool doInitialization(Module &M) override;
Kostya Serebryanye2a0e412012-02-13 22:50:51 +000086 static char ID; // Pass identification, replacement for typeid.
87
88 private:
Kostya Serebryany4b929da2012-11-29 09:54:21 +000089 void initializeCallbacks(Module &M);
Mehdi Aminia28d91d2015-03-10 02:37:25 +000090 bool instrumentLoadOrStore(Instruction *I, const DataLayout &DL);
91 bool instrumentAtomic(Instruction *I, const DataLayout &DL);
Kostya Serebryany463aa812013-03-28 11:21:13 +000092 bool instrumentMemIntrinsic(Instruction *I);
Mehdi Aminia28d91d2015-03-10 02:37:25 +000093 void chooseInstructionsToInstrument(SmallVectorImpl<Instruction *> &Local,
94 SmallVectorImpl<Instruction *> &All,
95 const DataLayout &DL);
Kostya Serebryany5ba61ac2012-04-10 22:29:17 +000096 bool addrPointsToConstantData(Value *Addr);
Mehdi Aminia28d91d2015-03-10 02:37:25 +000097 int getMemoryAccessFuncIndex(Value *Addr, const DataLayout &DL);
Kostya Serebryanybf2de802012-04-10 18:18:56 +000098
Kostya Serebryany463aa812013-03-28 11:21:13 +000099 Type *IntptrTy;
Kostya Serebryanya1259772012-04-27 07:31:53 +0000100 IntegerType *OrdTy;
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000101 // Callbacks to run-time library are computed in doInitialization.
Kostya Serebryanya1259772012-04-27 07:31:53 +0000102 Function *TsanFuncEntry;
103 Function *TsanFuncExit;
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000104 // Accesses sizes are powers of two: 1, 2, 4, 8, 16.
Kostya Serebryanya8531ee2012-02-14 00:52:07 +0000105 static const size_t kNumberOfAccessSizes = 5;
Kostya Serebryanya1259772012-04-27 07:31:53 +0000106 Function *TsanRead[kNumberOfAccessSizes];
107 Function *TsanWrite[kNumberOfAccessSizes];
Dmitry Vyukov91ffdec2015-01-27 20:19:17 +0000108 Function *TsanUnalignedRead[kNumberOfAccessSizes];
109 Function *TsanUnalignedWrite[kNumberOfAccessSizes];
Kostya Serebryanya1259772012-04-27 07:31:53 +0000110 Function *TsanAtomicLoad[kNumberOfAccessSizes];
111 Function *TsanAtomicStore[kNumberOfAccessSizes];
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000112 Function *TsanAtomicRMW[AtomicRMWInst::LAST_BINOP + 1][kNumberOfAccessSizes];
113 Function *TsanAtomicCAS[kNumberOfAccessSizes];
114 Function *TsanAtomicThreadFence;
115 Function *TsanAtomicSignalFence;
Kostya Serebryanya1259772012-04-27 07:31:53 +0000116 Function *TsanVptrUpdate;
Dmitry Vyukov55e63ef2013-03-22 08:51:22 +0000117 Function *TsanVptrLoad;
Kostya Serebryany463aa812013-03-28 11:21:13 +0000118 Function *MemmoveFn, *MemcpyFn, *MemsetFn;
Ismail Pazarbasi2d4ae9f2015-05-07 21:41:23 +0000119 Function *TsanCtorFunction;
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000120};
121} // namespace
122
123char ThreadSanitizer::ID = 0;
124INITIALIZE_PASS(ThreadSanitizer, "tsan",
125 "ThreadSanitizer: detects data races.",
126 false, false)
127
Kostya Serebryanya1259772012-04-27 07:31:53 +0000128const char *ThreadSanitizer::getPassName() const {
129 return "ThreadSanitizer";
130}
131
Alexey Samsonov6d8bab82014-06-02 18:08:27 +0000132FunctionPass *llvm::createThreadSanitizerPass() {
133 return new ThreadSanitizer();
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000134}
135
Kostya Serebryany4b929da2012-11-29 09:54:21 +0000136void ThreadSanitizer::initializeCallbacks(Module &M) {
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000137 IRBuilder<> IRB(M.getContext());
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000138 // Initialize the callbacks.
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000139 TsanFuncEntry = checkSanitizerInterfaceFunction(M.getOrInsertFunction(
Reid Kleckner971c3ea2014-11-13 22:55:19 +0000140 "__tsan_func_entry", IRB.getVoidTy(), IRB.getInt8PtrTy(), nullptr));
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000141 TsanFuncExit = checkSanitizerInterfaceFunction(
142 M.getOrInsertFunction("__tsan_func_exit", IRB.getVoidTy(), nullptr));
Kostya Serebryanya1259772012-04-27 07:31:53 +0000143 OrdTy = IRB.getInt32Ty();
Kostya Serebryanya8531ee2012-02-14 00:52:07 +0000144 for (size_t i = 0; i < kNumberOfAccessSizes; ++i) {
Yaron Kerendfb655f2015-08-15 19:06:14 +0000145 const unsigned ByteSize = 1U << i;
146 const unsigned BitSize = ByteSize * 8;
147 std::string ByteSizeStr = utostr(ByteSize);
148 std::string BitSizeStr = utostr(BitSize);
149 SmallString<32> ReadName("__tsan_read" + ByteSizeStr);
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000150 TsanRead[i] = checkSanitizerInterfaceFunction(M.getOrInsertFunction(
Reid Kleckner971c3ea2014-11-13 22:55:19 +0000151 ReadName, IRB.getVoidTy(), IRB.getInt8PtrTy(), nullptr));
Kostya Serebryanya1259772012-04-27 07:31:53 +0000152
Yaron Kerendfb655f2015-08-15 19:06:14 +0000153 SmallString<32> WriteName("__tsan_write" + ByteSizeStr);
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000154 TsanWrite[i] = checkSanitizerInterfaceFunction(M.getOrInsertFunction(
Reid Kleckner971c3ea2014-11-13 22:55:19 +0000155 WriteName, IRB.getVoidTy(), IRB.getInt8PtrTy(), nullptr));
Kostya Serebryanya1259772012-04-27 07:31:53 +0000156
Yaron Kerendfb655f2015-08-15 19:06:14 +0000157 SmallString<64> UnalignedReadName("__tsan_unaligned_read" + ByteSizeStr);
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000158 TsanUnalignedRead[i] =
159 checkSanitizerInterfaceFunction(M.getOrInsertFunction(
160 UnalignedReadName, IRB.getVoidTy(), IRB.getInt8PtrTy(), nullptr));
Dmitry Vyukov91ffdec2015-01-27 20:19:17 +0000161
Yaron Kerendfb655f2015-08-15 19:06:14 +0000162 SmallString<64> UnalignedWriteName("__tsan_unaligned_write" + ByteSizeStr);
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000163 TsanUnalignedWrite[i] =
164 checkSanitizerInterfaceFunction(M.getOrInsertFunction(
165 UnalignedWriteName, IRB.getVoidTy(), IRB.getInt8PtrTy(), nullptr));
Dmitry Vyukov91ffdec2015-01-27 20:19:17 +0000166
Kostya Serebryanya1259772012-04-27 07:31:53 +0000167 Type *Ty = Type::getIntNTy(M.getContext(), BitSize);
168 Type *PtrTy = Ty->getPointerTo();
Yaron Kerendfb655f2015-08-15 19:06:14 +0000169 SmallString<32> AtomicLoadName("__tsan_atomic" + BitSizeStr + "_load");
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000170 TsanAtomicLoad[i] = checkSanitizerInterfaceFunction(
171 M.getOrInsertFunction(AtomicLoadName, Ty, PtrTy, OrdTy, nullptr));
Kostya Serebryanya1259772012-04-27 07:31:53 +0000172
Yaron Kerendfb655f2015-08-15 19:06:14 +0000173 SmallString<32> AtomicStoreName("__tsan_atomic" + BitSizeStr + "_store");
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000174 TsanAtomicStore[i] = checkSanitizerInterfaceFunction(M.getOrInsertFunction(
175 AtomicStoreName, IRB.getVoidTy(), PtrTy, Ty, OrdTy, nullptr));
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000176
177 for (int op = AtomicRMWInst::FIRST_BINOP;
178 op <= AtomicRMWInst::LAST_BINOP; ++op) {
Craig Topperf40110f2014-04-25 05:29:35 +0000179 TsanAtomicRMW[op][i] = nullptr;
180 const char *NamePart = nullptr;
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000181 if (op == AtomicRMWInst::Xchg)
182 NamePart = "_exchange";
183 else if (op == AtomicRMWInst::Add)
184 NamePart = "_fetch_add";
185 else if (op == AtomicRMWInst::Sub)
186 NamePart = "_fetch_sub";
187 else if (op == AtomicRMWInst::And)
188 NamePart = "_fetch_and";
189 else if (op == AtomicRMWInst::Or)
190 NamePart = "_fetch_or";
191 else if (op == AtomicRMWInst::Xor)
192 NamePart = "_fetch_xor";
Dmitry Vyukova878e742012-11-27 08:09:25 +0000193 else if (op == AtomicRMWInst::Nand)
194 NamePart = "_fetch_nand";
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000195 else
196 continue;
197 SmallString<32> RMWName("__tsan_atomic" + itostr(BitSize) + NamePart);
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000198 TsanAtomicRMW[op][i] = checkSanitizerInterfaceFunction(
199 M.getOrInsertFunction(RMWName, Ty, PtrTy, Ty, OrdTy, nullptr));
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000200 }
201
Yaron Kerendfb655f2015-08-15 19:06:14 +0000202 SmallString<32> AtomicCASName("__tsan_atomic" + BitSizeStr +
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000203 "_compare_exchange_val");
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000204 TsanAtomicCAS[i] = checkSanitizerInterfaceFunction(M.getOrInsertFunction(
Reid Kleckner971c3ea2014-11-13 22:55:19 +0000205 AtomicCASName, Ty, PtrTy, Ty, Ty, OrdTy, OrdTy, nullptr));
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000206 }
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000207 TsanVptrUpdate = checkSanitizerInterfaceFunction(
208 M.getOrInsertFunction("__tsan_vptr_update", IRB.getVoidTy(),
209 IRB.getInt8PtrTy(), IRB.getInt8PtrTy(), nullptr));
210 TsanVptrLoad = checkSanitizerInterfaceFunction(M.getOrInsertFunction(
Reid Kleckner971c3ea2014-11-13 22:55:19 +0000211 "__tsan_vptr_read", IRB.getVoidTy(), IRB.getInt8PtrTy(), nullptr));
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000212 TsanAtomicThreadFence = checkSanitizerInterfaceFunction(M.getOrInsertFunction(
Reid Kleckner971c3ea2014-11-13 22:55:19 +0000213 "__tsan_atomic_thread_fence", IRB.getVoidTy(), OrdTy, nullptr));
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000214 TsanAtomicSignalFence = checkSanitizerInterfaceFunction(M.getOrInsertFunction(
Reid Kleckner971c3ea2014-11-13 22:55:19 +0000215 "__tsan_atomic_signal_fence", IRB.getVoidTy(), OrdTy, nullptr));
Kostya Serebryany463aa812013-03-28 11:21:13 +0000216
Ismail Pazarbasi198d6d52015-04-06 21:09:08 +0000217 MemmoveFn = checkSanitizerInterfaceFunction(
218 M.getOrInsertFunction("memmove", IRB.getInt8PtrTy(), IRB.getInt8PtrTy(),
219 IRB.getInt8PtrTy(), IntptrTy, nullptr));
220 MemcpyFn = checkSanitizerInterfaceFunction(
221 M.getOrInsertFunction("memcpy", IRB.getInt8PtrTy(), IRB.getInt8PtrTy(),
222 IRB.getInt8PtrTy(), IntptrTy, nullptr));
223 MemsetFn = checkSanitizerInterfaceFunction(
224 M.getOrInsertFunction("memset", IRB.getInt8PtrTy(), IRB.getInt8PtrTy(),
225 IRB.getInt32Ty(), IntptrTy, nullptr));
Kostya Serebryany4b929da2012-11-29 09:54:21 +0000226}
227
228bool ThreadSanitizer::doInitialization(Module &M) {
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000229 const DataLayout &DL = M.getDataLayout();
Ismail Pazarbasi2d4ae9f2015-05-07 21:41:23 +0000230 IntptrTy = DL.getIntPtrType(M.getContext());
231 std::tie(TsanCtorFunction, std::ignore) = createSanitizerCtorAndInitFunctions(
232 M, kTsanModuleCtorName, kTsanInitName, /*InitArgTypes=*/{},
233 /*InitArgs=*/{});
Kostya Serebryany4b929da2012-11-29 09:54:21 +0000234
Ismail Pazarbasi2d4ae9f2015-05-07 21:41:23 +0000235 appendToGlobalCtors(M, TsanCtorFunction, 0);
Kostya Serebryany4b929da2012-11-29 09:54:21 +0000236
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000237 return true;
238}
239
Kostya Serebryany5ba61ac2012-04-10 22:29:17 +0000240static bool isVtableAccess(Instruction *I) {
Duncan P. N. Exon Smithde36e802014-11-11 21:30:22 +0000241 if (MDNode *Tag = I->getMetadata(LLVMContext::MD_tbaa))
Manman Rend8c68b12013-09-06 22:47:05 +0000242 return Tag->isTBAAVtableAccess();
Kostya Serebryany5ba61ac2012-04-10 22:29:17 +0000243 return false;
244}
245
246bool ThreadSanitizer::addrPointsToConstantData(Value *Addr) {
247 // If this is a GEP, just analyze its pointer operand.
248 if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(Addr))
249 Addr = GEP->getPointerOperand();
250
251 if (GlobalVariable *GV = dyn_cast<GlobalVariable>(Addr)) {
252 if (GV->isConstant()) {
253 // Reads from constant globals can not race with any writes.
Kostya Serebryany5a4b7a22012-04-23 08:44:59 +0000254 NumOmittedReadsFromConstantGlobals++;
Kostya Serebryany5ba61ac2012-04-10 22:29:17 +0000255 return true;
256 }
Alexey Samsonovf54e3aa2012-08-30 13:47:13 +0000257 } else if (LoadInst *L = dyn_cast<LoadInst>(Addr)) {
Kostya Serebryany5ba61ac2012-04-10 22:29:17 +0000258 if (isVtableAccess(L)) {
259 // Reads from a vtable pointer can not race with any writes.
Kostya Serebryany5a4b7a22012-04-23 08:44:59 +0000260 NumOmittedReadsFromVtable++;
Kostya Serebryany5ba61ac2012-04-10 22:29:17 +0000261 return true;
262 }
263 }
264 return false;
265}
266
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000267// Instrumenting some of the accesses may be proven redundant.
268// Currently handled:
269// - read-before-write (within same BB, no calls between)
Dmitry Vyukov2e8d82e2015-02-12 09:55:28 +0000270// - not captured variables
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000271//
272// We do not handle some of the patterns that should not survive
273// after the classic compiler optimizations.
274// E.g. two reads from the same temp should be eliminated by CSE,
275// two writes should be eliminated by DSE, etc.
276//
277// 'Local' is a vector of insns within the same BB (no calls between).
278// 'All' is a vector of insns that will be instrumented.
Kostya Serebryanyae7188d2012-05-02 13:12:19 +0000279void ThreadSanitizer::chooseInstructionsToInstrument(
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000280 SmallVectorImpl<Instruction *> &Local, SmallVectorImpl<Instruction *> &All,
281 const DataLayout &DL) {
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000282 SmallSet<Value*, 8> WriteTargets;
283 // Iterate from the end.
284 for (SmallVectorImpl<Instruction*>::reverse_iterator It = Local.rbegin(),
285 E = Local.rend(); It != E; ++It) {
286 Instruction *I = *It;
287 if (StoreInst *Store = dyn_cast<StoreInst>(I)) {
288 WriteTargets.insert(Store->getPointerOperand());
289 } else {
290 LoadInst *Load = cast<LoadInst>(I);
Kostya Serebryany5ba61ac2012-04-10 22:29:17 +0000291 Value *Addr = Load->getPointerOperand();
292 if (WriteTargets.count(Addr)) {
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000293 // We will write to this temp, so no reason to analyze the read.
Kostya Serebryany5a4b7a22012-04-23 08:44:59 +0000294 NumOmittedReadsBeforeWrite++;
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000295 continue;
296 }
Kostya Serebryany5ba61ac2012-04-10 22:29:17 +0000297 if (addrPointsToConstantData(Addr)) {
298 // Addr points to some constant data -- it can not race with any writes.
299 continue;
300 }
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000301 }
Dmitry Vyukov2e8d82e2015-02-12 09:55:28 +0000302 Value *Addr = isa<StoreInst>(*I)
303 ? cast<StoreInst>(I)->getPointerOperand()
304 : cast<LoadInst>(I)->getPointerOperand();
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000305 if (isa<AllocaInst>(GetUnderlyingObject(Addr, DL)) &&
Dmitry Vyukov2e8d82e2015-02-12 09:55:28 +0000306 !PointerMayBeCaptured(Addr, true, true)) {
307 // The variable is addressable but not captured, so it cannot be
308 // referenced from a different thread and participate in a data race
309 // (see llvm/Analysis/CaptureTracking.h for details).
310 NumOmittedNonCaptured++;
311 continue;
312 }
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000313 All.push_back(I);
314 }
315 Local.clear();
316}
317
Kostya Serebryanya1259772012-04-27 07:31:53 +0000318static bool isAtomic(Instruction *I) {
319 if (LoadInst *LI = dyn_cast<LoadInst>(I))
320 return LI->isAtomic() && LI->getSynchScope() == CrossThread;
Kostya Serebryanyae7188d2012-05-02 13:12:19 +0000321 if (StoreInst *SI = dyn_cast<StoreInst>(I))
Kostya Serebryanya1259772012-04-27 07:31:53 +0000322 return SI->isAtomic() && SI->getSynchScope() == CrossThread;
Kostya Serebryanyae7188d2012-05-02 13:12:19 +0000323 if (isa<AtomicRMWInst>(I))
Kostya Serebryanya1259772012-04-27 07:31:53 +0000324 return true;
Kostya Serebryanyae7188d2012-05-02 13:12:19 +0000325 if (isa<AtomicCmpXchgInst>(I))
Kostya Serebryanya1259772012-04-27 07:31:53 +0000326 return true;
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000327 if (isa<FenceInst>(I))
328 return true;
Kostya Serebryanya1259772012-04-27 07:31:53 +0000329 return false;
330}
331
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000332bool ThreadSanitizer::runOnFunction(Function &F) {
Ismail Pazarbasi2d4ae9f2015-05-07 21:41:23 +0000333 // This is required to prevent instrumenting call to __tsan_init from within
334 // the module constructor.
335 if (&F == TsanCtorFunction)
336 return false;
Kostya Serebryany4b929da2012-11-29 09:54:21 +0000337 initializeCallbacks(*F.getParent());
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000338 SmallVector<Instruction*, 8> RetVec;
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000339 SmallVector<Instruction*, 8> AllLoadsAndStores;
340 SmallVector<Instruction*, 8> LocalLoadsAndStores;
Kostya Serebryanya1259772012-04-27 07:31:53 +0000341 SmallVector<Instruction*, 8> AtomicAccesses;
Kostya Serebryany463aa812013-03-28 11:21:13 +0000342 SmallVector<Instruction*, 8> MemIntrinCalls;
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000343 bool Res = false;
344 bool HasCalls = false;
Alexey Samsonov6d8bab82014-06-02 18:08:27 +0000345 bool SanitizeFunction = F.hasFnAttribute(Attribute::SanitizeThread);
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000346 const DataLayout &DL = F.getParent()->getDataLayout();
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000347
348 // Traverse all instructions, collect loads/stores/returns, check for calls.
Alexey Samsonova02e6642014-05-29 18:40:48 +0000349 for (auto &BB : F) {
350 for (auto &Inst : BB) {
351 if (isAtomic(&Inst))
352 AtomicAccesses.push_back(&Inst);
353 else if (isa<LoadInst>(Inst) || isa<StoreInst>(Inst))
354 LocalLoadsAndStores.push_back(&Inst);
355 else if (isa<ReturnInst>(Inst))
356 RetVec.push_back(&Inst);
357 else if (isa<CallInst>(Inst) || isa<InvokeInst>(Inst)) {
358 if (isa<MemIntrinsic>(Inst))
359 MemIntrinCalls.push_back(&Inst);
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000360 HasCalls = true;
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000361 chooseInstructionsToInstrument(LocalLoadsAndStores, AllLoadsAndStores,
362 DL);
Kostya Serebryanybf2de802012-04-10 18:18:56 +0000363 }
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000364 }
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000365 chooseInstructionsToInstrument(LocalLoadsAndStores, AllLoadsAndStores, DL);
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000366 }
367
368 // We have collected all loads and stores.
369 // FIXME: many of these accesses do not need to be checked for races
370 // (e.g. variables that do not escape, etc).
371
Alexey Samsonovd3828b82014-05-31 00:11:37 +0000372 // Instrument memory accesses only if we want to report bugs in the function.
373 if (ClInstrumentMemoryAccesses && SanitizeFunction)
Alexey Samsonova02e6642014-05-29 18:40:48 +0000374 for (auto Inst : AllLoadsAndStores) {
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000375 Res |= instrumentLoadOrStore(Inst, DL);
Kostya Serebryanyd23b18f2012-10-04 05:28:50 +0000376 }
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000377
Alexey Samsonovd3828b82014-05-31 00:11:37 +0000378 // Instrument atomic memory accesses in any case (they can be used to
379 // implement synchronization).
Kostya Serebryanyd23b18f2012-10-04 05:28:50 +0000380 if (ClInstrumentAtomics)
Alexey Samsonova02e6642014-05-29 18:40:48 +0000381 for (auto Inst : AtomicAccesses) {
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000382 Res |= instrumentAtomic(Inst, DL);
Kostya Serebryanyd23b18f2012-10-04 05:28:50 +0000383 }
Kostya Serebryanya1259772012-04-27 07:31:53 +0000384
Alexey Samsonovd3828b82014-05-31 00:11:37 +0000385 if (ClInstrumentMemIntrinsics && SanitizeFunction)
Alexey Samsonova02e6642014-05-29 18:40:48 +0000386 for (auto Inst : MemIntrinCalls) {
387 Res |= instrumentMemIntrinsic(Inst);
Kostya Serebryany463aa812013-03-28 11:21:13 +0000388 }
389
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000390 // Instrument function entry/exit points if there were instrumented accesses.
Kostya Serebryanyd23b18f2012-10-04 05:28:50 +0000391 if ((Res || HasCalls) && ClInstrumentFuncEntryExit) {
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000392 IRBuilder<> IRB(F.getEntryBlock().getFirstNonPHI());
393 Value *ReturnAddress = IRB.CreateCall(
394 Intrinsic::getDeclaration(F.getParent(), Intrinsic::returnaddress),
395 IRB.getInt32(0));
396 IRB.CreateCall(TsanFuncEntry, ReturnAddress);
Alexey Samsonova02e6642014-05-29 18:40:48 +0000397 for (auto RetInst : RetVec) {
398 IRBuilder<> IRBRet(RetInst);
David Blaikieff6409d2015-05-18 22:13:54 +0000399 IRBRet.CreateCall(TsanFuncExit, {});
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000400 }
Kostya Serebryany6f8a7762012-03-26 17:35:03 +0000401 Res = true;
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000402 }
403 return Res;
404}
405
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000406bool ThreadSanitizer::instrumentLoadOrStore(Instruction *I,
407 const DataLayout &DL) {
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000408 IRBuilder<> IRB(I);
409 bool IsWrite = isa<StoreInst>(*I);
410 Value *Addr = IsWrite
411 ? cast<StoreInst>(I)->getPointerOperand()
412 : cast<LoadInst>(I)->getPointerOperand();
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000413 int Idx = getMemoryAccessFuncIndex(Addr, DL);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000414 if (Idx < 0)
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000415 return false;
Kostya Serebryany6f8a7762012-03-26 17:35:03 +0000416 if (IsWrite && isVtableAccess(I)) {
Kostya Serebryanye36ae682012-07-05 09:07:31 +0000417 DEBUG(dbgs() << " VPTR : " << *I << "\n");
Kostya Serebryany6f8a7762012-03-26 17:35:03 +0000418 Value *StoredValue = cast<StoreInst>(I)->getValueOperand();
Kostya Serebryany08b9cf52013-12-02 08:07:15 +0000419 // StoredValue may be a vector type if we are storing several vptrs at once.
420 // In this case, just take the first element of the vector since this is
421 // enough to find vptr races.
422 if (isa<VectorType>(StoredValue->getType()))
423 StoredValue = IRB.CreateExtractElement(
424 StoredValue, ConstantInt::get(IRB.getInt32Ty(), 0));
Kostya Serebryany2460c3f2013-12-05 15:03:02 +0000425 if (StoredValue->getType()->isIntegerTy())
426 StoredValue = IRB.CreateIntToPtr(StoredValue, IRB.getInt8PtrTy());
Kostya Serebryanye36ae682012-07-05 09:07:31 +0000427 // Call TsanVptrUpdate.
David Blaikieff6409d2015-05-18 22:13:54 +0000428 IRB.CreateCall(TsanVptrUpdate,
429 {IRB.CreatePointerCast(Addr, IRB.getInt8PtrTy()),
430 IRB.CreatePointerCast(StoredValue, IRB.getInt8PtrTy())});
Kostya Serebryany5a4b7a22012-04-23 08:44:59 +0000431 NumInstrumentedVtableWrites++;
Kostya Serebryany6f8a7762012-03-26 17:35:03 +0000432 return true;
433 }
Dmitry Vyukov55e63ef2013-03-22 08:51:22 +0000434 if (!IsWrite && isVtableAccess(I)) {
435 IRB.CreateCall(TsanVptrLoad,
436 IRB.CreatePointerCast(Addr, IRB.getInt8PtrTy()));
437 NumInstrumentedVtableReads++;
438 return true;
439 }
Dmitry Vyukov91ffdec2015-01-27 20:19:17 +0000440 const unsigned Alignment = IsWrite
441 ? cast<StoreInst>(I)->getAlignment()
442 : cast<LoadInst>(I)->getAlignment();
443 Type *OrigTy = cast<PointerType>(Addr->getType())->getElementType();
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000444 const uint32_t TypeSize = DL.getTypeStoreSizeInBits(OrigTy);
Dmitry Vyukov91ffdec2015-01-27 20:19:17 +0000445 Value *OnAccessFunc = nullptr;
446 if (Alignment == 0 || Alignment >= 8 || (Alignment % (TypeSize / 8)) == 0)
447 OnAccessFunc = IsWrite ? TsanWrite[Idx] : TsanRead[Idx];
448 else
449 OnAccessFunc = IsWrite ? TsanUnalignedWrite[Idx] : TsanUnalignedRead[Idx];
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000450 IRB.CreateCall(OnAccessFunc, IRB.CreatePointerCast(Addr, IRB.getInt8PtrTy()));
Kostya Serebryany5a4b7a22012-04-23 08:44:59 +0000451 if (IsWrite) NumInstrumentedWrites++;
452 else NumInstrumentedReads++;
Kostya Serebryanye2a0e412012-02-13 22:50:51 +0000453 return true;
454}
Kostya Serebryanya1259772012-04-27 07:31:53 +0000455
456static ConstantInt *createOrdering(IRBuilder<> *IRB, AtomicOrdering ord) {
457 uint32_t v = 0;
458 switch (ord) {
Craig Topperd3c02f12015-01-05 10:15:49 +0000459 case NotAtomic: llvm_unreachable("unexpected atomic ordering!");
Kostya Serebryanya1259772012-04-27 07:31:53 +0000460 case Unordered: // Fall-through.
Dmitry Vyukov0044e382012-11-09 14:12:16 +0000461 case Monotonic: v = 0; break;
Dmitry Vyukov93e67b62012-11-26 14:55:26 +0000462 // case Consume: v = 1; break; // Not specified yet.
Dmitry Vyukov0044e382012-11-09 14:12:16 +0000463 case Acquire: v = 2; break;
464 case Release: v = 3; break;
465 case AcquireRelease: v = 4; break;
466 case SequentiallyConsistent: v = 5; break;
Kostya Serebryanya1259772012-04-27 07:31:53 +0000467 }
Dmitry Vyukov0044e382012-11-09 14:12:16 +0000468 return IRB->getInt32(v);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000469}
470
Kostya Serebryany463aa812013-03-28 11:21:13 +0000471// If a memset intrinsic gets inlined by the code gen, we will miss races on it.
472// So, we either need to ensure the intrinsic is not inlined, or instrument it.
473// We do not instrument memset/memmove/memcpy intrinsics (too complicated),
474// instead we simply replace them with regular function calls, which are then
475// intercepted by the run-time.
476// Since tsan is running after everyone else, the calls should not be
477// replaced back with intrinsics. If that becomes wrong at some point,
478// we will need to call e.g. __tsan_memset to avoid the intrinsics.
479bool ThreadSanitizer::instrumentMemIntrinsic(Instruction *I) {
480 IRBuilder<> IRB(I);
481 if (MemSetInst *M = dyn_cast<MemSetInst>(I)) {
David Blaikieff6409d2015-05-18 22:13:54 +0000482 IRB.CreateCall(
483 MemsetFn,
484 {IRB.CreatePointerCast(M->getArgOperand(0), IRB.getInt8PtrTy()),
485 IRB.CreateIntCast(M->getArgOperand(1), IRB.getInt32Ty(), false),
486 IRB.CreateIntCast(M->getArgOperand(2), IntptrTy, false)});
Kostya Serebryany463aa812013-03-28 11:21:13 +0000487 I->eraseFromParent();
488 } else if (MemTransferInst *M = dyn_cast<MemTransferInst>(I)) {
David Blaikieff6409d2015-05-18 22:13:54 +0000489 IRB.CreateCall(
490 isa<MemCpyInst>(M) ? MemcpyFn : MemmoveFn,
491 {IRB.CreatePointerCast(M->getArgOperand(0), IRB.getInt8PtrTy()),
492 IRB.CreatePointerCast(M->getArgOperand(1), IRB.getInt8PtrTy()),
493 IRB.CreateIntCast(M->getArgOperand(2), IntptrTy, false)});
Kostya Serebryany463aa812013-03-28 11:21:13 +0000494 I->eraseFromParent();
495 }
496 return false;
497}
498
Dmitry Vyukov12b5cb92012-11-26 11:36:19 +0000499// Both llvm and ThreadSanitizer atomic operations are based on C++11/C1x
Alp Tokercb402912014-01-24 17:20:08 +0000500// standards. For background see C++11 standard. A slightly older, publicly
Dmitry Vyukov12b5cb92012-11-26 11:36:19 +0000501// available draft of the standard (not entirely up-to-date, but close enough
502// for casual browsing) is available here:
Matt Beaumont-Gay000a3952012-11-26 16:27:22 +0000503// http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2011/n3242.pdf
Dmitry Vyukov12b5cb92012-11-26 11:36:19 +0000504// The following page contains more background information:
505// http://www.hpl.hp.com/personal/Hans_Boehm/c++mm/
506
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000507bool ThreadSanitizer::instrumentAtomic(Instruction *I, const DataLayout &DL) {
Kostya Serebryanya1259772012-04-27 07:31:53 +0000508 IRBuilder<> IRB(I);
509 if (LoadInst *LI = dyn_cast<LoadInst>(I)) {
510 Value *Addr = LI->getPointerOperand();
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000511 int Idx = getMemoryAccessFuncIndex(Addr, DL);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000512 if (Idx < 0)
513 return false;
Yaron Kerendfb655f2015-08-15 19:06:14 +0000514 const unsigned ByteSize = 1U << Idx;
515 const unsigned BitSize = ByteSize * 8;
Kostya Serebryanya1259772012-04-27 07:31:53 +0000516 Type *Ty = Type::getIntNTy(IRB.getContext(), BitSize);
Micah Villmow51e72462012-10-24 17:25:11 +0000517 Type *PtrTy = Ty->getPointerTo();
Kostya Serebryanya1259772012-04-27 07:31:53 +0000518 Value *Args[] = {IRB.CreatePointerCast(Addr, PtrTy),
519 createOrdering(&IRB, LI->getOrdering())};
Craig Toppere1d12942014-08-27 05:25:25 +0000520 CallInst *C = CallInst::Create(TsanAtomicLoad[Idx], Args);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000521 ReplaceInstWithInst(I, C);
522
523 } else if (StoreInst *SI = dyn_cast<StoreInst>(I)) {
524 Value *Addr = SI->getPointerOperand();
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000525 int Idx = getMemoryAccessFuncIndex(Addr, DL);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000526 if (Idx < 0)
527 return false;
Yaron Kerendfb655f2015-08-15 19:06:14 +0000528 const unsigned ByteSize = 1U << Idx;
529 const unsigned BitSize = ByteSize * 8;
Kostya Serebryanya1259772012-04-27 07:31:53 +0000530 Type *Ty = Type::getIntNTy(IRB.getContext(), BitSize);
Micah Villmow51e72462012-10-24 17:25:11 +0000531 Type *PtrTy = Ty->getPointerTo();
Kostya Serebryanya1259772012-04-27 07:31:53 +0000532 Value *Args[] = {IRB.CreatePointerCast(Addr, PtrTy),
533 IRB.CreateIntCast(SI->getValueOperand(), Ty, false),
534 createOrdering(&IRB, SI->getOrdering())};
Craig Toppere1d12942014-08-27 05:25:25 +0000535 CallInst *C = CallInst::Create(TsanAtomicStore[Idx], Args);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000536 ReplaceInstWithInst(I, C);
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000537 } else if (AtomicRMWInst *RMWI = dyn_cast<AtomicRMWInst>(I)) {
538 Value *Addr = RMWI->getPointerOperand();
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000539 int Idx = getMemoryAccessFuncIndex(Addr, DL);
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000540 if (Idx < 0)
541 return false;
542 Function *F = TsanAtomicRMW[RMWI->getOperation()][Idx];
Craig Topperf40110f2014-04-25 05:29:35 +0000543 if (!F)
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000544 return false;
Yaron Kerendfb655f2015-08-15 19:06:14 +0000545 const unsigned ByteSize = 1U << Idx;
546 const unsigned BitSize = ByteSize * 8;
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000547 Type *Ty = Type::getIntNTy(IRB.getContext(), BitSize);
548 Type *PtrTy = Ty->getPointerTo();
549 Value *Args[] = {IRB.CreatePointerCast(Addr, PtrTy),
550 IRB.CreateIntCast(RMWI->getValOperand(), Ty, false),
551 createOrdering(&IRB, RMWI->getOrdering())};
Craig Toppere1d12942014-08-27 05:25:25 +0000552 CallInst *C = CallInst::Create(F, Args);
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000553 ReplaceInstWithInst(I, C);
554 } else if (AtomicCmpXchgInst *CASI = dyn_cast<AtomicCmpXchgInst>(I)) {
555 Value *Addr = CASI->getPointerOperand();
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000556 int Idx = getMemoryAccessFuncIndex(Addr, DL);
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000557 if (Idx < 0)
558 return false;
Yaron Kerendfb655f2015-08-15 19:06:14 +0000559 const unsigned ByteSize = 1U << Idx;
560 const unsigned BitSize = ByteSize * 8;
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000561 Type *Ty = Type::getIntNTy(IRB.getContext(), BitSize);
562 Type *PtrTy = Ty->getPointerTo();
563 Value *Args[] = {IRB.CreatePointerCast(Addr, PtrTy),
564 IRB.CreateIntCast(CASI->getCompareOperand(), Ty, false),
565 IRB.CreateIntCast(CASI->getNewValOperand(), Ty, false),
Tim Northovere94a5182014-03-11 10:48:52 +0000566 createOrdering(&IRB, CASI->getSuccessOrdering()),
567 createOrdering(&IRB, CASI->getFailureOrdering())};
Tim Northover420a2162014-06-13 14:24:07 +0000568 CallInst *C = IRB.CreateCall(TsanAtomicCAS[Idx], Args);
569 Value *Success = IRB.CreateICmpEQ(C, CASI->getCompareOperand());
570
571 Value *Res = IRB.CreateInsertValue(UndefValue::get(CASI->getType()), C, 0);
572 Res = IRB.CreateInsertValue(Res, Success, 1);
573
574 I->replaceAllUsesWith(Res);
575 I->eraseFromParent();
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000576 } else if (FenceInst *FI = dyn_cast<FenceInst>(I)) {
577 Value *Args[] = {createOrdering(&IRB, FI->getOrdering())};
578 Function *F = FI->getSynchScope() == SingleThread ?
579 TsanAtomicSignalFence : TsanAtomicThreadFence;
Craig Toppere1d12942014-08-27 05:25:25 +0000580 CallInst *C = CallInst::Create(F, Args);
Dmitry Vyukov92b9e1d2012-11-09 12:55:36 +0000581 ReplaceInstWithInst(I, C);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000582 }
583 return true;
584}
585
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000586int ThreadSanitizer::getMemoryAccessFuncIndex(Value *Addr,
587 const DataLayout &DL) {
Kostya Serebryanya1259772012-04-27 07:31:53 +0000588 Type *OrigPtrTy = Addr->getType();
589 Type *OrigTy = cast<PointerType>(OrigPtrTy)->getElementType();
590 assert(OrigTy->isSized());
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000591 uint32_t TypeSize = DL.getTypeStoreSizeInBits(OrigTy);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000592 if (TypeSize != 8 && TypeSize != 16 &&
593 TypeSize != 32 && TypeSize != 64 && TypeSize != 128) {
594 NumAccessesWithBadSize++;
595 // Ignore all unusual sizes.
596 return -1;
597 }
Michael J. Spencerdf1ecbd72013-05-24 22:23:49 +0000598 size_t Idx = countTrailingZeros(TypeSize / 8);
Kostya Serebryanya1259772012-04-27 07:31:53 +0000599 assert(Idx < kNumberOfAccessSizes);
600 return Idx;
601}