You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by kp...@apache.org on 2023/01/30 20:15:14 UTC

[tvm] branch main updated: [LLVM] Remove call to EmitDebugLocation from AddAliasInfo (#13872)

This is an automated email from the ASF dual-hosted git repository.

kparzysz pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git


The following commit(s) were added to refs/heads/main by this push:
     new c81aaa852c [LLVM] Remove call to EmitDebugLocation from AddAliasInfo (#13872)
c81aaa852c is described below

commit c81aaa852c5b9de72f8dbc5fdaa088e7e35bedb7
Author: Krzysztof Parzyszek <kp...@quicinc.com>
AuthorDate: Mon Jan 30 14:15:07 2023 -0600

    [LLVM] Remove call to EmitDebugLocation from AddAliasInfo (#13872)
    
    This function only creates alias metadata, so there isn't anything
    for it to create debug location information for. If `index` is used
    in executable code, the debug location should be emitted then.
---
 src/target/llvm/codegen_llvm.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/target/llvm/codegen_llvm.cc b/src/target/llvm/codegen_llvm.cc
index 2182ecfa51..dcca337320 100644
--- a/src/target/llvm/codegen_llvm.cc
+++ b/src/target/llvm/codegen_llvm.cc
@@ -557,7 +557,6 @@ llvm::Type* CodeGenLLVM::GetLLVMType(const PrimExpr& expr) const {
 //
 void CodeGenLLVM::AddAliasInfo(llvm::Instruction* inst, const VarNode* buffer_var, PrimExpr index,
                                DataType access_dtype) {
-  EmitDebugLocation(index->span);
   if (alias_var_set_.count(buffer_var) != 0) {
     // Mark all possibly aliased pointer as same type.
     llvm::MDNode* meta = md_tbaa_alias_set_;