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