summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com>2019-08-13 01:07:27 +0000
committerStanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com>2019-08-13 01:07:27 +0000
commit438315bf69c7dae68edd5c99f9672fffdc442a79 (patch)
tree57d2cc7a8ddec8223a26f40132dd89fa2775e851
parent6e18c7f8d4b5d9b4ea1c9590289489ab9115bff0 (diff)
downloadbcm5719-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.cpp8
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));
OpenPOWER on IntegriCloud