From 1202f559bdeebb62dbba29ee05ffafbfecd57d92 Mon Sep 17 00:00:00 2001 From: Arthur Eubanks Date: Tue, 25 May 2021 12:36:25 -0700 Subject: [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 --- llvm/lib/Bitcode/Reader/BitcodeReader.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp') 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(I)->setVolatile(IsVol); InstructionList.push_back(I); -- cgit v1.1