r261657 - Remove an unnecessary workaround introduced in r259975. (NFC)
Adrian Prantl via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 23 11:23:19 PST 2016
I will carefully revert the revert of this commit on top of the current trunk. Thanks for the notice.
-- adrian
> On Feb 23, 2016, at 11:15 AM, Hans Wennborg <hans at chromium.org> wrote:
>
> I'm sorry, but I had to revert this in order to revert r261634. r261657
>
> On Tue, Feb 23, 2016 at 9:13 AM, Adrian Prantl via cfe-commits
> <cfe-commits at lists.llvm.org> wrote:
>> Author: adrian
>> Date: Tue Feb 23 11:13:47 2016
>> New Revision: 261657
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=261657&view=rev
>> Log:
>> Remove an unnecessary workaround introduced in r259975. (NFC)
>>
>> Now that LLVM r259973 allows replacing a temporary type with another
>> temporary we can rely on the original implementation.
>>
>> It is possible for enums to be created as part of
>> their own declcontext. In this case a FwdDecl will be created
>> twice. This doesn't cause a problem because both FwdDecls are
>> entered into the ReplaceMap: finalize() will replace the first
>> FwdDecl with the second and then replace the second with
>> complete type.
>>
>> Thanks to echristo for pointing this out.
>>
>> Modified:
>> cfe/trunk/lib/CodeGen/CGDebugInfo.cpp
>>
>> Modified: cfe/trunk/lib/CodeGen/CGDebugInfo.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGDebugInfo.cpp?rev=261657&r1=261656&r2=261657&view=diff
>> ==============================================================================
>> --- cfe/trunk/lib/CodeGen/CGDebugInfo.cpp (original)
>> +++ cfe/trunk/lib/CodeGen/CGDebugInfo.cpp Tue Feb 23 11:13:47 2016
>> @@ -2071,25 +2071,22 @@ llvm::DIType *CGDebugInfo::CreateEnumTyp
>> // If this is just a forward declaration, construct an appropriately
>> // marked node and just return it.
>> if (isImportedFromModule || !ED->getDefinition()) {
>> + // Note that it is possible for enums to be created as part of
>> + // their own declcontext. In this case a FwdDecl will be created
>> + // twice. This doesn't cause a problem because both FwdDecls are
>> + // entered into the ReplaceMap: finalize() will replace the first
>> + // FwdDecl with the second and then replace the second with
>> + // complete type.
>> + llvm::DIScope *EDContext = getDeclContextDescriptor(ED);
>> llvm::DIFile *DefUnit = getOrCreateFile(ED->getLocation());
>> -
>> - // It is possible for enums to be created as part of their own
>> - // declcontext. We need to cache a placeholder to avoid the type being
>> - // created twice before hitting the cache.
>> llvm::TempDIScope TmpContext(DBuilder.createReplaceableCompositeType(
>> llvm::dwarf::DW_TAG_enumeration_type, "", TheCU, DefUnit, 0));
>>
>> unsigned Line = getLineNumber(ED->getLocation());
>> StringRef EDName = ED->getName();
>> llvm::DIType *RetTy = DBuilder.createReplaceableCompositeType(
>> - llvm::dwarf::DW_TAG_enumeration_type, EDName, TmpContext.get(), DefUnit,
>> - Line, 0, Size, Align, llvm::DINode::FlagFwdDecl, FullName);
>> -
>> - // Cache the enum type so it is available when building the declcontext
>> - // and replace the declcontect with the real thing.
>> - TypeCache[Ty].reset(RetTy);
>> - TmpContext->replaceAllUsesWith(
>> - getDeclarationLexicalScope(*ED, QualType(Ty, 0)));
>> + llvm::dwarf::DW_TAG_enumeration_type, EDName, EDContext, DefUnit, Line,
>> + 0, Size, Align, llvm::DINode::FlagFwdDecl, FullName);
>>
>> ReplaceMap.emplace_back(
>> std::piecewise_construct, std::make_tuple(Ty),
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
More information about the cfe-commits
mailing list