[all-commits] [llvm/llvm-project] 5dd86a: [WebAssembly] Add TargetInstrInfo::getCalleeOperand

Heejin Ahn via All-commits all-commits at lists.llvm.org
Wed May 26 11:44:38 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5dd86aadf0b014913bd35bb7435808eb081bc049
      https://github.com/llvm/llvm-project/commit/5dd86aadf0b014913bd35bb7435808eb081bc049
  Author: Heejin Ahn <aheejin at gmail.com>
  Date:   2021-05-26 (Wed, 26 May 2021)

  Changed paths:
    M llvm/include/llvm/CodeGen/TargetInstrInfo.h
    M llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
    M llvm/lib/Target/WebAssembly/Utils/WebAssemblyUtilities.cpp
    M llvm/lib/Target/WebAssembly/WebAssemblyInstrInfo.cpp
    M llvm/lib/Target/WebAssembly/WebAssemblyInstrInfo.h
    A llvm/test/DebugInfo/WebAssembly/call-site.ll

  Log Message:
  -----------
  [WebAssembly] Add TargetInstrInfo::getCalleeOperand

DwarfDebug unconditionally assumes for all call instructions the 0th
operand is the callee operand, which seems to be true for other targets,
but not for WebAssembly. This adds `TargetInstrInfo::getCallOperand`
method whose default implementation returns `getOperand(0)` and makes
WebAssembly overrides it to use its own utility method to get the callee
operand.

This also fixes an existing bug in `WebAssembly::getCalleeOp`, which was
uncovered by this CL.

Reviewed By: dschuff, djtodoro

Differential Revision: https://reviews.llvm.org/D102978




More information about the All-commits mailing list