diff options
author | Stanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com> | 2019-08-13 01:07:27 +0000 |
---|---|---|
committer | Stanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com> | 2019-08-13 01:07:27 +0000 |
commit | 438315bf69c7dae68edd5c99f9672fffdc442a79 (patch) | |
tree | 57d2cc7a8ddec8223a26f40132dd89fa2775e851 | |
parent | 6e18c7f8d4b5d9b4ea1c9590289489ab9115bff0 (diff) | |
download | bcm5719-llvm-438315bf69c7dae68edd5c99f9672fffdc442a79.tar.gz bcm5719-llvm-438315bf69c7dae68edd5c99f9672fffdc442a79.zip |
[AMDGPU] Fix msan failure in printf lowering
llvm-svn: 368645
-rw-r--r-- | llvm/lib/Target/AMDGPU/AMDGPUPrintfRuntimeBinding.cpp | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUPrintfRuntimeBinding.cpp b/llvm/lib/Target/AMDGPU/AMDGPUPrintfRuntimeBinding.cpp index 85cf902a4ed..022e8a060ed 100644 --- a/llvm/lib/Target/AMDGPU/AMDGPUPrintfRuntimeBinding.cpp +++ b/llvm/lib/Target/AMDGPU/AMDGPUPrintfRuntimeBinding.cpp @@ -537,8 +537,8 @@ bool AMDGPUPrintfRuntimeBinding::lowerPrintfForGpu(Module &M) { } else { WhatToStore.push_back(Arg); } - for (auto W : WhatToStore) { - Value *TheBtCast = W; + for (unsigned I = 0, E = WhatToStore.size(); I != E; ++I) { + Value *TheBtCast = WhatToStore[I]; unsigned ArgSize = TD->getTypeAllocSizeInBits(TheBtCast->getType()) / 8; SmallVector<Value *, 1> BuffOffset; @@ -551,9 +551,7 @@ bool AMDGPUPrintfRuntimeBinding::lowerPrintfForGpu(Module &M) { LLVM_DEBUG(dbgs() << "inserting store to printf buffer:\n" << *StBuff << '\n'); (void)StBuff; - ++W; - if (W == *WhatToStore.end() && - ArgCount + 1 == CI->getNumArgOperands()) + if (I + 1 == E && ArgCount + 1 == CI->getNumArgOperands()) break; BufferIdx = dyn_cast<GetElementPtrInst>(GetElementPtrInst::Create( nullptr, BufferIdx, BuffOffset, "PrintBuffNextPtr", Brnch)); |