xref: /llvm-project/llvm/lib/Target/WebAssembly/WebAssemblyDebugFixup.cpp (revision 43570a2841e2a8f1efd00503beee751cc1e72513)
12b7fe086SWouter van Oortmerssen //===-- WebAssemblyDebugFixup.cpp - Debug Fixup ------------------===//
22b7fe086SWouter van Oortmerssen //
32b7fe086SWouter van Oortmerssen // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
42b7fe086SWouter van Oortmerssen // See https://llvm.org/LICENSE.txt for license information.
52b7fe086SWouter van Oortmerssen // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
62b7fe086SWouter van Oortmerssen //
72b7fe086SWouter van Oortmerssen //===----------------------------------------------------------------------===//
82b7fe086SWouter van Oortmerssen ///
92b7fe086SWouter van Oortmerssen /// \file
102b7fe086SWouter van Oortmerssen /// Several prior passes may "stackify" registers, here we ensure any references
112b7fe086SWouter van Oortmerssen /// in such registers in debug_value instructions become stack relative also.
122b7fe086SWouter van Oortmerssen /// This is done in a separate pass such that not all previous passes need to
132b7fe086SWouter van Oortmerssen /// track stack depth when values get stackified.
142b7fe086SWouter van Oortmerssen ///
152b7fe086SWouter van Oortmerssen //===----------------------------------------------------------------------===//
162b7fe086SWouter van Oortmerssen 
172b7fe086SWouter van Oortmerssen #include "MCTargetDesc/WebAssemblyMCTargetDesc.h"
182b7fe086SWouter van Oortmerssen #include "WebAssembly.h"
192b7fe086SWouter van Oortmerssen #include "WebAssemblyMachineFunctionInfo.h"
202b7fe086SWouter van Oortmerssen #include "WebAssemblySubtarget.h"
21984dc4b9SReid Kleckner #include "WebAssemblyUtilities.h"
222b7fe086SWouter van Oortmerssen #include "llvm/CodeGen/MachineFrameInfo.h"
232b7fe086SWouter van Oortmerssen #include "llvm/CodeGen/MachineFunction.h"
242b7fe086SWouter van Oortmerssen #include "llvm/CodeGen/MachineInstrBuilder.h"
252b7fe086SWouter van Oortmerssen #include "llvm/CodeGen/Passes.h"
262b7fe086SWouter van Oortmerssen #include "llvm/Support/Debug.h"
272b7fe086SWouter van Oortmerssen #include "llvm/Support/raw_ostream.h"
282b7fe086SWouter van Oortmerssen using namespace llvm;
292b7fe086SWouter van Oortmerssen 
302b7fe086SWouter van Oortmerssen #define DEBUG_TYPE "wasm-debug-fixup"
312b7fe086SWouter van Oortmerssen 
322b7fe086SWouter van Oortmerssen namespace {
332b7fe086SWouter van Oortmerssen class WebAssemblyDebugFixup final : public MachineFunctionPass {
342b7fe086SWouter van Oortmerssen   StringRef getPassName() const override { return "WebAssembly Debug Fixup"; }
352b7fe086SWouter van Oortmerssen 
362b7fe086SWouter van Oortmerssen   void getAnalysisUsage(AnalysisUsage &AU) const override {
372b7fe086SWouter van Oortmerssen     AU.setPreservesCFG();
382b7fe086SWouter van Oortmerssen     MachineFunctionPass::getAnalysisUsage(AU);
392b7fe086SWouter van Oortmerssen   }
402b7fe086SWouter van Oortmerssen 
412b7fe086SWouter van Oortmerssen   bool runOnMachineFunction(MachineFunction &MF) override;
422b7fe086SWouter van Oortmerssen 
432b7fe086SWouter van Oortmerssen public:
442b7fe086SWouter van Oortmerssen   static char ID; // Pass identification, replacement for typeid
452b7fe086SWouter van Oortmerssen   WebAssemblyDebugFixup() : MachineFunctionPass(ID) {}
462b7fe086SWouter van Oortmerssen };
472b7fe086SWouter van Oortmerssen } // end anonymous namespace
482b7fe086SWouter van Oortmerssen 
492b7fe086SWouter van Oortmerssen char WebAssemblyDebugFixup::ID = 0;
502b7fe086SWouter van Oortmerssen INITIALIZE_PASS(
512b7fe086SWouter van Oortmerssen     WebAssemblyDebugFixup, DEBUG_TYPE,
522b7fe086SWouter van Oortmerssen     "Ensures debug_value's that have been stackified become stack relative",
532b7fe086SWouter van Oortmerssen     false, false)
542b7fe086SWouter van Oortmerssen 
552b7fe086SWouter van Oortmerssen FunctionPass *llvm::createWebAssemblyDebugFixup() {
562b7fe086SWouter van Oortmerssen   return new WebAssemblyDebugFixup();
572b7fe086SWouter van Oortmerssen }
582b7fe086SWouter van Oortmerssen 
59981a28f7SHeejin Ahn // At this very end of the compilation pipeline, if any DBG_VALUEs with
60981a28f7SHeejin Ahn // registers remain, it means they are dangling info which we failed to update
61981a28f7SHeejin Ahn // when their corresponding def instruction was transformed/moved/splitted etc.
62981a28f7SHeejin Ahn // Because Wasm cannot access values in LLVM virtual registers in the debugger,
63981a28f7SHeejin Ahn // these dangling DBG_VALUEs in effect kill the effect of any previous DBG_VALUE
64981a28f7SHeejin Ahn // associated with the variable, which will appear as "optimized out".
654f2401f4SHeejin Ahn static void setDanglingDebugValuesUndef(MachineBasicBlock &MBB,
66981a28f7SHeejin Ahn                                         const TargetInstrInfo *TII) {
67981a28f7SHeejin Ahn   for (auto &MI : llvm::make_early_inc_range(MBB)) {
68981a28f7SHeejin Ahn     if (MI.isDebugValue() && MI.getDebugOperand(0).isReg() &&
69981a28f7SHeejin Ahn         !MI.isUndefDebugValue()) {
704f2401f4SHeejin Ahn       LLVM_DEBUG(dbgs() << "Warning: dangling DBG_VALUE set to undef: " << MI
71981a28f7SHeejin Ahn                         << "\n");
724f2401f4SHeejin Ahn       MI.setDebugValueUndef();
73981a28f7SHeejin Ahn     }
74981a28f7SHeejin Ahn   }
75981a28f7SHeejin Ahn }
76981a28f7SHeejin Ahn 
772b7fe086SWouter van Oortmerssen bool WebAssemblyDebugFixup::runOnMachineFunction(MachineFunction &MF) {
782b7fe086SWouter van Oortmerssen   LLVM_DEBUG(dbgs() << "********** Debug Fixup **********\n"
792b7fe086SWouter van Oortmerssen                        "********** Function: "
802b7fe086SWouter van Oortmerssen                     << MF.getName() << '\n');
812b7fe086SWouter van Oortmerssen 
822b7fe086SWouter van Oortmerssen   WebAssemblyFunctionInfo &MFI = *MF.getInfo<WebAssemblyFunctionInfo>();
832b7fe086SWouter van Oortmerssen   const auto *TII = MF.getSubtarget<WebAssemblySubtarget>().getInstrInfo();
842b7fe086SWouter van Oortmerssen 
852b7fe086SWouter van Oortmerssen   struct StackElem {
862b7fe086SWouter van Oortmerssen     unsigned Reg;
872b7fe086SWouter van Oortmerssen     MachineInstr *DebugValue;
882b7fe086SWouter van Oortmerssen   };
892b7fe086SWouter van Oortmerssen   std::vector<StackElem> Stack;
902b7fe086SWouter van Oortmerssen   for (MachineBasicBlock &MBB : MF) {
912b7fe086SWouter van Oortmerssen     // We may insert into this list.
922b7fe086SWouter van Oortmerssen     for (auto MII = MBB.begin(); MII != MBB.end(); ++MII) {
932b7fe086SWouter van Oortmerssen       MachineInstr &MI = *MII;
942b7fe086SWouter van Oortmerssen       if (MI.isDebugValue()) {
952b7fe086SWouter van Oortmerssen         auto &MO = MI.getOperand(0);
962b7fe086SWouter van Oortmerssen         // Also check if not a $noreg: likely a DBG_VALUE we just inserted.
972b7fe086SWouter van Oortmerssen         if (MO.isReg() && MO.getReg().isValid() &&
982b7fe086SWouter van Oortmerssen             MFI.isVRegStackified(MO.getReg())) {
992b7fe086SWouter van Oortmerssen           // Found a DBG_VALUE with a stackified register we will
1002b7fe086SWouter van Oortmerssen           // change into a stack operand.
1012b7fe086SWouter van Oortmerssen           // Search for register rather than assume it is on top (which it
1022b7fe086SWouter van Oortmerssen           // typically is if it appears right after the def), since
1032b7fe086SWouter van Oortmerssen           // DBG_VALUE's may shift under some circumstances.
10410e2e7deSWouter van Oortmerssen           for (auto &Elem : reverse(Stack)) {
1052b7fe086SWouter van Oortmerssen             if (MO.getReg() == Elem.Reg) {
10610e2e7deSWouter van Oortmerssen               auto Depth = static_cast<unsigned>(&Elem - &Stack[0]);
107*cd3667d1SCraig Topper               LLVM_DEBUG(dbgs() << "Debug Value VReg " << printReg(MO.getReg())
1082b7fe086SWouter van Oortmerssen                                 << " -> Stack Relative " << Depth << "\n");
1092b7fe086SWouter van Oortmerssen               MO.ChangeToTargetIndex(WebAssembly::TI_OPERAND_STACK, Depth);
1102b7fe086SWouter van Oortmerssen               // Save the DBG_VALUE instruction that defined this stackified
1112b7fe086SWouter van Oortmerssen               // variable since later we need it to construct another one on
1122b7fe086SWouter van Oortmerssen               // pop.
1132b7fe086SWouter van Oortmerssen               Elem.DebugValue = &MI;
1142b7fe086SWouter van Oortmerssen               break;
1152b7fe086SWouter van Oortmerssen             }
1162b7fe086SWouter van Oortmerssen           }
1172b7fe086SWouter van Oortmerssen           // If the Reg was not found, we have a DBG_VALUE outside of its
1182b7fe086SWouter van Oortmerssen           // def-use range, and we leave it unmodified as reg, which means
1192b7fe086SWouter van Oortmerssen           // it will be culled later.
1202b7fe086SWouter van Oortmerssen         }
1212b7fe086SWouter van Oortmerssen       } else {
1222b7fe086SWouter van Oortmerssen         // Track stack depth.
1232b7fe086SWouter van Oortmerssen         for (MachineOperand &MO : reverse(MI.explicit_uses())) {
1242b7fe086SWouter van Oortmerssen           if (MO.isReg() && MFI.isVRegStackified(MO.getReg())) {
1252b7fe086SWouter van Oortmerssen             auto Prev = Stack.back();
1262b7fe086SWouter van Oortmerssen             Stack.pop_back();
1272b7fe086SWouter van Oortmerssen             assert(Prev.Reg == MO.getReg() &&
1282b7fe086SWouter van Oortmerssen                    "WebAssemblyDebugFixup: Pop: Register not matched!");
12971fbfb49SHeejin Ahn             // We should not put a DBG_VALUE after a terminator; debug ranges
13071fbfb49SHeejin Ahn             // are terminated at the end of a BB anyway.
13171fbfb49SHeejin Ahn             if (Prev.DebugValue && !MI.isTerminator()) {
1322b7fe086SWouter van Oortmerssen               // This stackified reg is a variable that started life at
1332b7fe086SWouter van Oortmerssen               // Prev.DebugValue, so now that we're popping it we must insert
1342b7fe086SWouter van Oortmerssen               // a $noreg DBG_VALUE for the variable to end it, right after
1352b7fe086SWouter van Oortmerssen               // the current instruction.
1362b7fe086SWouter van Oortmerssen               BuildMI(*Prev.DebugValue->getParent(), std::next(MII),
13771fbfb49SHeejin Ahn                       Prev.DebugValue->getDebugLoc(),
13871fbfb49SHeejin Ahn                       TII->get(WebAssembly::DBG_VALUE), false, Register(),
13971fbfb49SHeejin Ahn                       Prev.DebugValue->getOperand(2).getMetadata(),
1402b7fe086SWouter van Oortmerssen                       Prev.DebugValue->getOperand(3).getMetadata());
1412b7fe086SWouter van Oortmerssen             }
1422b7fe086SWouter van Oortmerssen           }
1432b7fe086SWouter van Oortmerssen         }
1442b7fe086SWouter van Oortmerssen         for (MachineOperand &MO : MI.defs()) {
1452b7fe086SWouter van Oortmerssen           if (MO.isReg() && MFI.isVRegStackified(MO.getReg())) {
1462b7fe086SWouter van Oortmerssen             Stack.push_back({MO.getReg(), nullptr});
1472b7fe086SWouter van Oortmerssen           }
1482b7fe086SWouter van Oortmerssen         }
1492b7fe086SWouter van Oortmerssen       }
1502b7fe086SWouter van Oortmerssen     }
1512b7fe086SWouter van Oortmerssen     assert(Stack.empty() &&
1522b7fe086SWouter van Oortmerssen            "WebAssemblyDebugFixup: Stack not empty at end of basic block!");
153981a28f7SHeejin Ahn 
1544f2401f4SHeejin Ahn     setDanglingDebugValuesUndef(MBB, TII);
1552b7fe086SWouter van Oortmerssen   }
1562b7fe086SWouter van Oortmerssen 
1572b7fe086SWouter van Oortmerssen   return true;
1582b7fe086SWouter van Oortmerssen }
159