From 5e7f98071a2c6f3178a5b083ef8ebe462af6ab85 Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Tue, 25 Nov 2014 13:19:46 +0000 Subject: [PATCH] Remove a bit of duplicated code. Exactly the same checks are present in areTypesIsomorphic. This might have been a premature performance optimization. I cannot reproduce any slowdown with this patch. llvm-svn: 222758 --- llvm/lib/Linker/LinkModules.cpp | 8 -------- 1 file changed, 8 deletions(-) diff --git a/llvm/lib/Linker/LinkModules.cpp b/llvm/lib/Linker/LinkModules.cpp index 5040738..44b4bce 100644 --- a/llvm/lib/Linker/LinkModules.cpp +++ b/llvm/lib/Linker/LinkModules.cpp @@ -96,14 +96,6 @@ private: } void TypeMapTy::addTypeMapping(Type *DstTy, Type *SrcTy) { - Type *&Entry = MappedTypes[SrcTy]; - if (Entry) return; - - if (DstTy == SrcTy) { - Entry = DstTy; - return; - } - // Check to see if these types are recursively isomorphic and establish a // mapping between them if so. if (areTypesIsomorphic(DstTy, SrcTy)) { -- 2.7.4