search for: fdecl

...( Type::getVoidTy( context ), types, false ); // m1 Function* f = Function::Create( ft, GlobalValue::ExternalLinkage, "f", m1 ); b = BasicBlock::Create( context, "b", f ); ReturnInst::Create( context, b ); verifyModule(*m1); // m2 Function* fDecl = Function::Create( ft, GlobalValue::ExternalLinkage, "f", m2 ); Function* h = Function::Create( ft, GlobalValue::ExternalLinkage, "h", m2 ); vector<Value*> args; args.push_back( h->arg_begin() ); b = BasicBlock::Create( context, "b", h )...

...; > // m1 > Function* f = Function::Create( ft, GlobalValue::ExternalLinkage, "f", > m1 ); > > b = BasicBlock::Create( context, "b", f ); > ReturnInst::Create( context, b ); > > verifyModule(*m1); > > // m2 > Function* fDecl = Function::Create( ft, GlobalValue::ExternalLinkage, > "f", m2 ); > Function* h = Function::Create( ft, GlobalValue::ExternalLinkage, "h", > m2 ); > > vector<Value*> args; > args.push_back( h->arg_begin() ); > > b = BasicBlock::C...

Hi Dmitry, the direct call patch seems like a good start; however, by falling through to the F->replaceUsesExceptBlockAddr(FDecl) at the bottom of LowerTypeTestsModule::importFunction() I believe it will replace all uses of a function (for a definition outside the TU it's defined in) with the direct function reference instead of just direct calls. Also, I believe the logic for replaceDirectCalls() should be more restrict...

Replacing direct calls to jump table entries with calls to real targets recovers most of the performance loss in my benchmark. Dealing with link order files is a bit cumbersome though: I can’t keep both, say “foo” and “foo.cfi” next to each other in link order, because if both exist, the linker reorders the jump table next to the real function. This is not what we want when the goal is to get rid

.../plain; charset="utf-8" > > > > > On Apr 30, 2018, at 5:43 PM, Vlad Tsyrklevich <vlad at tsyrklevich.net> > wrote: > > > > Hi Dmitry, the direct call patch seems like a good start; however, by > falling through to the F->replaceUsesExceptBlockAddr(FDecl) at the bottom > of LowerTypeTestsModule::importFunction() I believe it will replace all > uses of a function (for a definition outside the TU it's defined in) with > the direct function reference instead of just direct calls. > > You’re right, it was meant to bail after direct c...