diff options
| author | Arthur Eubanks <aeubanks@google.com> | 2021-05-25 12:36:25 -0700 |
|---|---|---|
| committer | Arthur Eubanks <aeubanks@google.com> | 2021-05-25 20:16:21 -0700 |
| commit | 1202f559bdeebb62dbba29ee05ffafbfecd57d92 (patch) | |
| tree | ba5ae3ef0ca5e0561342082bedebd678bd0aa97e /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | 564eb20e0deecd173a7b990dcfd0e57fb045c522 (diff) | |
| download | llvm-1202f559bdeebb62dbba29ee05ffafbfecd57d92.zip llvm-1202f559bdeebb62dbba29ee05ffafbfecd57d92.tar.gz llvm-1202f559bdeebb62dbba29ee05ffafbfecd57d92.tar.bz2 | |
[OpaquePtr] Make atomicrmw work with opaque pointers
FullTy is only necessary when we need to figure out what type an
instruction works with given a pointer's pointee type. However, we just
end up using the value operand's type, so FullTy isn't necessary.
Reviewed By: dblaikie
Differential Revision: https://reviews.llvm.org/D102788
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
| -rw-r--r-- | llvm/lib/Bitcode/Reader/BitcodeReader.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Bitcode/Reader/BitcodeReader.cpp b/llvm/lib/Bitcode/Reader/BitcodeReader.cpp index ad85066..58b1b2b 100644 --- a/llvm/lib/Bitcode/Reader/BitcodeReader.cpp +++ b/llvm/lib/Bitcode/Reader/BitcodeReader.cpp @@ -5251,9 +5251,11 @@ Error BitcodeReader::parseFunctionBody(Function *F) { if (popValue(Record, OpNum, NextValueNo, getPointerElementFlatType(FullTy), Val)) return error("Invalid record"); + FullTy = getPointerElementFlatType(FullTy); } else { if (getValueTypePair(Record, OpNum, NextValueNo, Val)) return error("Invalid record"); + FullTy = Val->getType(); } if (!(NumRecords == (OpNum + 4) || NumRecords == (OpNum + 5))) @@ -5286,7 +5288,6 @@ Error BitcodeReader::parseFunctionBody(Function *F) { Align(TheModule->getDataLayout().getTypeStoreSize(Val->getType())); I = new AtomicRMWInst(Operation, Ptr, Val, *Alignment, Ordering, SSID); - FullTy = getPointerElementFlatType(FullTy); cast<AtomicRMWInst>(I)->setVolatile(IsVol); InstructionList.push_back(I); |