[Autobackout][FuncReg]Revert of change: 8d1656d

 Enable trampoline insertion pass by default

TrampolineInsertion pass must be enabled by default to make VISA and
binary linking possible.

Author: Anton Sidorenko

IGC/VectorCompiler/include/vc/Utils/GenX/KernelInfo.h

@@ -9,9 +9,7 @@ SPDX-License-Identifier: MIT
 #ifndef VC_UTILS_GENX_KERNELINFO_H
 #define VC_UTILS_GENX_KERNELINFO_H
 
-#include "vc/InternalIntrinsics/InternalIntrinsics.h"
 #include "vc/Utils/GenX/InternalMetadata.h"
-#include "vc/Utils/GenX/Intrinsics.h"
 #include "vc/Utils/GenX/RegCategory.h"
 
 #include "Probe/Assertion.h"
@@ -90,18 +88,13 @@ inline bool isIndirect(const llvm::Function *F) {
 // structure types is fixed for intrinsics.
   if (llvm::GenXIntrinsic::isAnyNonTrivialIntrinsic(F))
     return false;
-  if (vc::InternalIntrinsic::isInternalNonTrivialIntrinsic(F))
-    return false;
-  if (vc::isKernel(F))
-    return false;
-  if (vc::isEmulationFunction(*F))
-    return false;
-  if (!F->hasAddressTaken() && F->hasLocalLinkage())
-    return false;
+  // FIXME: The condition of which function is considered to be indirectly
+  // called will be changed soon.
+  bool IsIndirect = F->hasAddressTaken();
   IGC_ASSERT_MESSAGE(
-      requiresStackCall(F),
+      !IsIndirect || requiresStackCall(F),
       "The indirectly-called function is expected to be a stack call");
-  return true;
+  return IsIndirect;
 }
 
 inline bool isIndirect(const llvm::Function &F) { return isIndirect(&F); }

IGC/VectorCompiler/lib/GenXCodeGen/GenXCisaBuilder.cpp

@@ -5431,6 +5431,10 @@ void GenXKernelBuilder::buildInlineAsm(CallInst *CI) {
 void GenXKernelBuilder::buildCall(CallInst *CI, const DstOpndDesc &DstDesc) {
   LLVM_DEBUG(dbgs() << CI << "\n");
   Function *Callee = CI->getCalledFunction();
+  IGC_ASSERT_MESSAGE(
+      !Callee || !Callee->isDeclaration(),
+      "Currently VC backend does not support modules with external functions");
+
   if (!Callee || vc::requiresStackCall(Callee)) {
     if (UseNewStackBuilder)
       buildStackCallLight(CI, DstDesc);

IGC/VectorCompiler/lib/GenXCodeGen/GenXGlobalValueLowering.cpp

@@ -208,8 +208,10 @@ void GenXGlobalValueLowering::fillWorkListForGVInstUse(Use &GVUse) {
                          GenXIntrinsic::genx_gaddr,
                      "llvm.gaddr must be inserted by this pass, but someone "
                      "inserted it before");
-  // Do not skip direct calls as an indirectly-called function can be called
-  // only indirectly. See GenXTrampolineInsertion
+  // Skipping direct calls. Relocating will make them indirect.
+  if (isa<Function>(ConstWithGV) && isa<CallInst>(Usr) &&
+      cast<CallInst>(Usr)->getCalledFunction() == cast<Function>(ConstWithGV))
+    return;
   Function *Func = Usr->getParent()->getParent();
   WorkList[Func].Users.insert({Usr, GVUse.getOperandNo()});
   WorkList[Func].Replacement[ConstWithGV] = nullptr;

IGC/VectorCompiler/lib/GenXOpts/CMTrans/GenXTrampolineInsertion.cpp

@@ -52,7 +52,6 @@ SPDX-License-Identifier: MIT
 
 #include "llvmWrapper/IR/Value.h"
 #include "vc/GenXOpts/GenXOpts.h"
-#include "vc/InternalIntrinsics/InternalIntrinsics.h"
 #include "vc/Support/BackendConfig.h"
 #include "vc/Utils/GenX/Intrinsics.h"
 #include "vc/Utils/GenX/KernelInfo.h"
@@ -68,7 +67,7 @@ using namespace llvm;
 
 static cl::opt<bool> EnableTrampolineInsertion(
     "vc-enable-trampoline-insertion",
-    llvm::cl::desc("Enable/disable GenXTrampolineInsertion"), cl::init(true),
+    llvm::cl::desc("Enable/disable GenXTrampolineInsertion"), cl::init(false),
     cl::Hidden);
 
 namespace {
@@ -100,8 +99,6 @@ class GenXTrampolineInsertion : public ModulePass,
 void GenXTrampolineInsertion::visitFunction(Function &F) {
   if (GenXIntrinsic::isAnyNonTrivialIntrinsic(&F))
     return;
-  if (vc::InternalIntrinsic::isInternalNonTrivialIntrinsic(&F))
-    return;
   if (vc::isEmulationFunction(F))
     return;
   if (vc::isKernel(&F))