[Codegen][LLVM] Remove cast to i8* in builtin::address_of

[Lunderberg]

This cast was initially added when CreateBufferPtr did not include the cast to the appropriate address space, and should no longer be necessary. Removing the cast will not harm LLVM configurations with opaque pointers (default in LLVM
15+), and avoids type mismatches when using LLVM configurations that use typed pointers.

[tvm-bot]

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

• No users to tag found in teams: codegen, llvm _{See #10317 for details}

_{Generated by tvm-bot}

[tvm-bot]

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

• No users to tag found in teams: codegen, llvm _{See #10317 for details}

_{Generated by tvm-bot}

[Lunderberg]

This caused failures in llvm::verifyModule on a dev branch, in which the generated LLVM specified a function call to a function with a known signature. This doesn't usually result in verification failures for external function calls, because CodeGenLLVM::CreateCallExtern uses the call site itself to define the signature of each llvm::FunctionType.