[PATCH] D49887: [DebugInfo] Add support for DWARF5 call site-related attributes

[Heejin Ahn via Phabricator via llvm-commits]

aheejin added inline comments.
Herald added subscribers: sstefan1, cmtice, ormris, MaskRay.
Herald added a reviewer: jdoerfert.
Herald added a reviewer: jhenderson.
Herald added a project: LLVM.


================
Comment at: llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp:536
+      // If this is a direct call, find the callee's subprogram.
+      const MachineOperand &CalleeOp = MI.getOperand(0);
+      if (!CalleeOp.isGlobal())
----------------
Sorry for the late question, but is the first operand always the callee in a call instruction? That's not true for WebAssembly, but I'm wondering if WebAssembly is an exception and this holds for all other targets.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D49887/new/

https://reviews.llvm.org/D49887