[AMDGPU] Change ASAN init/fini kernels linkage to external.

[pvellien]

Summary

HSA runtime fails to find the symbols for Init and Fini kernels as they mark with internal linkage, changing the linkage to external to fix those errors.

[yaxunl]

need a test

[pvellien]

Hi @yaxunl It will helpful to know what kind of test is expected to test the linkage type of generated init/fini kernels. The related test-cases for init/fini kernels are added in this patch - https://reviews.llvm.org/D105682

[yaxunl]

In D110054#3009960, @pvellien wrote:

Hi @yaxunl It will helpful to know what kind of test is expected to test the linkage type of generated init/fini kernels. The related test-cases for init/fini kernels are added in this patch - https://reviews.llvm.org/D105682

Need a test to demonstrate and confirm the linkage change.

[pvellien]

Added test.

[foad]

Nit: first line of commit message should be more like [AMDGPU] Change ASAN init/fini kernels to use external linkage (with a capital letter but without a period). And there's no need to repeat this in the body of the commit message. Either leave it blank, or add some text that explains the reason for the change.

[pvellien]

@foad thanks.

[yaxunl]

LGTM. Thanks.

[pvellien]

Hi, The failures in the debian builds are not related to this patch changes. I don't have a commit access to llvm trunk, could any of you please land this patch on my behalf. Thanks a lot for your time.

[gandhi21299]

I can help commit this patch. Can you please rebase your work with the latest main branch and upload the patch again, @pvellien ? Thanks

[pvellien]

Rebase on top.

[pvellien]

In D110054#3024859, @gandhi21299 wrote:

I can help commit this patch. Can you please rebase your work with the latest main branch and upload the patch again, @pvellien ? Thanks

thanks @gandhi21299

[gandhi21299]

error: lib/Target/AMDGPU/AMDGPUCtorDtorLowering.cpp: No such file or directory
error: test/CodeGen/AMDGPU/lower-ctor-dtor.ll: No such file or directory
error: test/CodeGen/AMDGPU/lower-multiple-ctor-dtor.ll: No such file or directory

These errors were issued again despite llvm-project being up to date on my system. I manually implemented the changes. I am running all tests anyways and will push it once the tests pass.

llvm/lib/Target/AMDGPU/AMDGPUCtorDtorLowering.cpp

 public:
   Function *createInitOrFiniKernelFunction(Module &M, bool IsCtor) {
     StringRef InitOrFiniKernelName = "amdgcn.device.init";
     if (!IsCtor)
       InitOrFiniKernelName = "amdgcn.device.fini";
 
     Function *InitOrFiniKernel = Function::createWithDefaultAttr(
         FunctionType::get(Type::getVoidTy(M.getContext()), false),
-        GlobalValue::InternalLinkage, 0, InitOrFiniKernelName, &M);
+        GlobalValue::ExternalLinkage, 0, InitOrFiniKernelName, &M);
     BasicBlock *InitOrFiniKernelBB =
         BasicBlock::Create(M.getContext(), "", InitOrFiniKernel);
     ReturnInst::Create(M.getContext(), InitOrFiniKernelBB);
 
     InitOrFiniKernel->setCallingConv(CallingConv::AMDGPU_KERNEL);
     if (IsCtor)
       InitOrFiniKernel->addFnAttr("device-init");
     else