From: Evan Cheng Date: Mon, 8 Dec 2008 06:52:43 +0000 (+0000) Subject: Perform cheap checks first. X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=dda346aa717f12e9e99587d66c825429760a4e1d;p=oota-llvm.git Perform cheap checks first. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@60689 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/X86/X86Subtarget.cpp b/lib/Target/X86/X86Subtarget.cpp index 583fe8a0fbe..c9d14d3d5d7 100644 --- a/lib/Target/X86/X86Subtarget.cpp +++ b/lib/Target/X86/X86Subtarget.cpp @@ -40,6 +40,8 @@ bool X86Subtarget::GVRequiresExtraLoad(const GlobalValue* GV, if (TM.getRelocationModel() != Reloc::Static && TM.getCodeModel() != CodeModel::Large) { if (isTargetDarwin()) { + if (!isDirectCall) + return false; bool isDecl = GV->isDeclaration() && !GV->hasNotBeenReadFromBitcode(); if (GV->hasHiddenVisibility() && (Is64Bit || (!isDecl && !GV->hasCommonLinkage()))) @@ -47,7 +49,7 @@ bool X86Subtarget::GVRequiresExtraLoad(const GlobalValue* GV, // target is x86-64 or the symbol is definitely defined in the current // translation unit. return false; - return !isDirectCall && (isDecl || GV->mayBeOverridden()); + return isDecl || GV->mayBeOverridden(); } else if (isTargetELF()) { // Extra load is needed for all externally visible. if (isDirectCall)