blob: f3d4dd68170b845a1e0b4ba458bcebfd0a7a6e76 [file] [log] [blame]
Heejin Ahn4934f762018-06-25 01:07:11 +00001//=== WebAssemblyLateEHPrepare.cpp - WebAssembly Exception Preparation -===//
2//
Chandler Carruth2946cd72019-01-19 08:50:56 +00003// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4// See https://llvm.org/LICENSE.txt for license information.
5// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
Heejin Ahn4934f762018-06-25 01:07:11 +00006//
7//===----------------------------------------------------------------------===//
8///
9/// \file
10/// \brief Does various transformations for exception handling.
11///
12//===----------------------------------------------------------------------===//
13
14#include "MCTargetDesc/WebAssemblyMCTargetDesc.h"
15#include "WebAssembly.h"
16#include "WebAssemblySubtarget.h"
17#include "WebAssemblyUtilities.h"
Heejin Ahnd6f48782019-01-30 03:21:57 +000018#include "llvm/ADT/SmallSet.h"
Heejin Ahn4934f762018-06-25 01:07:11 +000019#include "llvm/CodeGen/MachineInstrBuilder.h"
20#include "llvm/CodeGen/WasmEHFuncInfo.h"
21#include "llvm/MC/MCAsmInfo.h"
22using namespace llvm;
23
Heejin Ahn8b49b6b2019-03-13 00:37:31 +000024#define DEBUG_TYPE "wasm-late-eh-prepare"
Heejin Ahn4934f762018-06-25 01:07:11 +000025
26namespace {
27class WebAssemblyLateEHPrepare final : public MachineFunctionPass {
28 StringRef getPassName() const override {
Heejin Ahnd6f48782019-01-30 03:21:57 +000029 return "WebAssembly Late Prepare Exception";
Heejin Ahn4934f762018-06-25 01:07:11 +000030 }
31
32 bool runOnMachineFunction(MachineFunction &MF) override;
Heejin Ahnd6f48782019-01-30 03:21:57 +000033 bool addCatches(MachineFunction &MF);
Heejin Ahnb47a18c2019-03-16 04:46:05 +000034 bool replaceFuncletReturns(MachineFunction &MF);
35 bool removeUnnecessaryUnreachables(MachineFunction &MF);
Heejin Ahnd6f48782019-01-30 03:21:57 +000036 bool addExceptionExtraction(MachineFunction &MF);
Heejin Ahn0bb98652019-01-30 22:44:45 +000037 bool restoreStackPointer(MachineFunction &MF);
Heejin Ahn4934f762018-06-25 01:07:11 +000038
39public:
40 static char ID; // Pass identification, replacement for typeid
41 WebAssemblyLateEHPrepare() : MachineFunctionPass(ID) {}
42};
43} // end anonymous namespace
44
45char WebAssemblyLateEHPrepare::ID = 0;
46INITIALIZE_PASS(WebAssemblyLateEHPrepare, DEBUG_TYPE,
Heejin Ahna93e7262018-08-17 00:12:04 +000047 "WebAssembly Late Exception Preparation", false, false)
Heejin Ahn4934f762018-06-25 01:07:11 +000048
49FunctionPass *llvm::createWebAssemblyLateEHPrepare() {
50 return new WebAssemblyLateEHPrepare();
51}
52
53// Returns the nearest EH pad that dominates this instruction. This does not use
54// dominator analysis; it just does BFS on its predecessors until arriving at an
55// EH pad. This assumes valid EH scopes so the first EH pad it arrives in all
56// possible search paths should be the same.
57// Returns nullptr in case it does not find any EH pad in the search, or finds
58// multiple different EH pads.
Heejin Ahn095796a2018-11-16 00:47:18 +000059static MachineBasicBlock *getMatchingEHPad(MachineInstr *MI) {
Heejin Ahn4934f762018-06-25 01:07:11 +000060 MachineFunction *MF = MI->getParent()->getParent();
61 SmallVector<MachineBasicBlock *, 2> WL;
62 SmallPtrSet<MachineBasicBlock *, 2> Visited;
63 WL.push_back(MI->getParent());
64 MachineBasicBlock *EHPad = nullptr;
65 while (!WL.empty()) {
66 MachineBasicBlock *MBB = WL.pop_back_val();
67 if (Visited.count(MBB))
68 continue;
69 Visited.insert(MBB);
70 if (MBB->isEHPad()) {
71 if (EHPad && EHPad != MBB)
72 return nullptr;
73 EHPad = MBB;
74 continue;
75 }
76 if (MBB == &MF->front())
77 return nullptr;
78 WL.append(MBB->pred_begin(), MBB->pred_end());
79 }
80 return EHPad;
81}
82
Heejin Ahn095796a2018-11-16 00:47:18 +000083// Erase the specified BBs if the BB does not have any remaining predecessors,
84// and also all its dead children.
Benjamin Kramer83996e42018-08-08 10:13:19 +000085template <typename Container>
Heejin Ahn095796a2018-11-16 00:47:18 +000086static void eraseDeadBBsAndChildren(const Container &MBBs) {
Benjamin Kramer83996e42018-08-08 10:13:19 +000087 SmallVector<MachineBasicBlock *, 8> WL(MBBs.begin(), MBBs.end());
Heejin Ahn4934f762018-06-25 01:07:11 +000088 while (!WL.empty()) {
89 MachineBasicBlock *MBB = WL.pop_back_val();
Heejin Ahn095796a2018-11-16 00:47:18 +000090 if (!MBB->pred_empty())
91 continue;
Heejin Ahnb68d5912018-10-04 21:03:35 +000092 SmallVector<MachineBasicBlock *, 4> Succs(MBB->succ_begin(),
93 MBB->succ_end());
94 WL.append(MBB->succ_begin(), MBB->succ_end());
95 for (auto *Succ : Succs)
Heejin Ahn4934f762018-06-25 01:07:11 +000096 MBB->removeSuccessor(Succ);
Heejin Ahn4934f762018-06-25 01:07:11 +000097 MBB->eraseFromParent();
98 }
99}
100
101bool WebAssemblyLateEHPrepare::runOnMachineFunction(MachineFunction &MF) {
Heejin Ahn569f0902019-01-09 23:05:21 +0000102 LLVM_DEBUG(dbgs() << "********** Late EH Prepare **********\n"
103 "********** Function: "
104 << MF.getName() << '\n');
105
Heejin Ahn4934f762018-06-25 01:07:11 +0000106 if (MF.getTarget().getMCAsmInfo()->getExceptionHandlingType() !=
107 ExceptionHandling::Wasm)
108 return false;
109
110 bool Changed = false;
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000111 if (MF.getFunction().hasPersonalityFn()) {
112 Changed |= addCatches(MF);
113 Changed |= replaceFuncletReturns(MF);
114 }
Heejin Ahn095796a2018-11-16 00:47:18 +0000115 Changed |= removeUnnecessaryUnreachables(MF);
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000116 if (MF.getFunction().hasPersonalityFn()) {
117 Changed |= addExceptionExtraction(MF);
118 Changed |= restoreStackPointer(MF);
119 }
Heejin Ahn4934f762018-06-25 01:07:11 +0000120 return Changed;
121}
122
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000123// Add catch instruction to beginning of catchpads and cleanuppads.
124bool WebAssemblyLateEHPrepare::addCatches(MachineFunction &MF) {
Heejin Ahn095796a2018-11-16 00:47:18 +0000125 bool Changed = false;
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000126 const auto &TII = *MF.getSubtarget<WebAssemblySubtarget>().getInstrInfo();
127 MachineRegisterInfo &MRI = MF.getRegInfo();
Heejin Ahn095796a2018-11-16 00:47:18 +0000128 for (auto &MBB : MF) {
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000129 if (MBB.isEHPad()) {
Heejin Ahn095796a2018-11-16 00:47:18 +0000130 Changed = true;
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000131 auto InsertPos = MBB.begin();
132 if (InsertPos->isEHLabel()) // EH pad starts with an EH label
133 ++InsertPos;
Daniel Sanders05c145d2019-08-12 22:40:45 +0000134 Register DstReg = MRI.createVirtualRegister(&WebAssembly::EXNREFRegClass);
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000135 BuildMI(MBB, InsertPos, MBB.begin()->getDebugLoc(),
136 TII.get(WebAssembly::CATCH), DstReg);
Heejin Ahn095796a2018-11-16 00:47:18 +0000137 }
138 }
Heejin Ahn095796a2018-11-16 00:47:18 +0000139 return Changed;
140}
141
Heejin Ahn4934f762018-06-25 01:07:11 +0000142bool WebAssemblyLateEHPrepare::replaceFuncletReturns(MachineFunction &MF) {
143 bool Changed = false;
144 const auto &TII = *MF.getSubtarget<WebAssemblySubtarget>().getInstrInfo();
Heejin Ahn4934f762018-06-25 01:07:11 +0000145
146 for (auto &MBB : MF) {
147 auto Pos = MBB.getFirstTerminator();
148 if (Pos == MBB.end())
149 continue;
150 MachineInstr *TI = &*Pos;
151
152 switch (TI->getOpcode()) {
153 case WebAssembly::CATCHRET: {
154 // Replace a catchret with a branch
155 MachineBasicBlock *TBB = TI->getOperand(0).getMBB();
156 if (!MBB.isLayoutSuccessor(TBB))
157 BuildMI(MBB, TI, TI->getDebugLoc(), TII.get(WebAssembly::BR))
158 .addMBB(TBB);
159 TI->eraseFromParent();
160 Changed = true;
161 break;
162 }
Heejin Ahn66ce4192019-03-16 05:38:57 +0000163 case WebAssembly::CLEANUPRET:
164 case WebAssembly::RETHROW_IN_CATCH: {
165 // Replace a cleanupret/rethrow_in_catch with a rethrow
166 auto *EHPad = getMatchingEHPad(TI);
167 auto CatchPos = EHPad->begin();
168 if (CatchPos->isEHLabel()) // EH pad starts with an EH label
169 ++CatchPos;
170 MachineInstr *Catch = &*CatchPos;
Daniel Sanders05c145d2019-08-12 22:40:45 +0000171 Register ExnReg = Catch->getOperand(0).getReg();
Heejin Ahn66ce4192019-03-16 05:38:57 +0000172 BuildMI(MBB, TI, TI->getDebugLoc(), TII.get(WebAssembly::RETHROW))
173 .addReg(ExnReg);
Heejin Ahn4934f762018-06-25 01:07:11 +0000174 TI->eraseFromParent();
175 Changed = true;
176 break;
177 }
178 }
179 }
180 return Changed;
181}
182
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000183bool WebAssemblyLateEHPrepare::removeUnnecessaryUnreachables(
184 MachineFunction &MF) {
Heejin Ahn4934f762018-06-25 01:07:11 +0000185 bool Changed = false;
Heejin Ahn4934f762018-06-25 01:07:11 +0000186 for (auto &MBB : MF) {
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000187 for (auto &MI : MBB) {
188 if (MI.getOpcode() != WebAssembly::THROW &&
189 MI.getOpcode() != WebAssembly::RETHROW)
190 continue;
Heejin Ahn4934f762018-06-25 01:07:11 +0000191 Changed = true;
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000192
193 // The instruction after the throw should be an unreachable or a branch to
194 // another BB that should eventually lead to an unreachable. Delete it
195 // because throw itself is a terminator, and also delete successors if
196 // any.
197 MBB.erase(std::next(MI.getIterator()), MBB.end());
198 SmallVector<MachineBasicBlock *, 8> Succs(MBB.succ_begin(),
199 MBB.succ_end());
200 for (auto *Succ : Succs)
Heejin Ahn66ce4192019-03-16 05:38:57 +0000201 if (!Succ->isEHPad())
202 MBB.removeSuccessor(Succ);
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000203 eraseDeadBBsAndChildren(Succs);
Heejin Ahn4934f762018-06-25 01:07:11 +0000204 }
205 }
Heejin Ahnb47a18c2019-03-16 04:46:05 +0000206
Heejin Ahn4934f762018-06-25 01:07:11 +0000207 return Changed;
208}
209
Heejin Ahnd6f48782019-01-30 03:21:57 +0000210// Wasm uses 'br_on_exn' instruction to check the tag of an exception. It takes
Heejin Ahn9f96a582019-07-15 22:49:25 +0000211// exnref type object returned by 'catch', and branches to the destination if it
212// matches a given tag. We currently use __cpp_exception symbol to represent the
213// tag for all C++ exceptions.
Heejin Ahnd6f48782019-01-30 03:21:57 +0000214//
215// block $l (result i32)
216// ...
Heejin Ahn9f96a582019-07-15 22:49:25 +0000217// ;; exnref $e is on the stack at this point
Heejin Ahnd6f48782019-01-30 03:21:57 +0000218// br_on_exn $l $e ;; branch to $l with $e's arguments
219// ...
220// end
221// ;; Here we expect the extracted values are on top of the wasm value stack
222// ... Handle exception using values ...
223//
Heejin Ahn9f96a582019-07-15 22:49:25 +0000224// br_on_exn takes an exnref object and branches if it matches the given tag.
225// There can be multiple br_on_exn instructions if we want to match for another
226// tag, but for now we only test for __cpp_exception tag, and if it does not
227// match, i.e., it is a foreign exception, we rethrow it.
Heejin Ahnd6f48782019-01-30 03:21:57 +0000228//
229// In the destination BB that's the target of br_on_exn, extracted exception
230// values (in C++'s case a single i32, which represents an exception pointer)
231// are placed on top of the wasm stack. Because we can't model wasm stack in
232// LLVM instruction, we use 'extract_exception' pseudo instruction to retrieve
233// it. The pseudo instruction will be deleted later.
234bool WebAssemblyLateEHPrepare::addExceptionExtraction(MachineFunction &MF) {
Heejin Ahn4934f762018-06-25 01:07:11 +0000235 const auto &TII = *MF.getSubtarget<WebAssemblySubtarget>().getInstrInfo();
Heejin Ahn831efe02019-08-11 06:24:07 +0000236 MachineRegisterInfo &MRI = MF.getRegInfo();
Heejin Ahn4934f762018-06-25 01:07:11 +0000237 auto *EHInfo = MF.getWasmEHFuncInfo();
Heejin Ahnd6f48782019-01-30 03:21:57 +0000238 SmallVector<MachineInstr *, 16> ExtractInstrs;
Heejin Ahnd2a56ac2019-02-26 04:08:49 +0000239 SmallVector<MachineInstr *, 8> ToDelete;
Heejin Ahnd6f48782019-01-30 03:21:57 +0000240 for (auto &MBB : MF) {
Heejin Ahn4934f762018-06-25 01:07:11 +0000241 for (auto &MI : MBB) {
Heejin Ahnd6f48782019-01-30 03:21:57 +0000242 if (MI.getOpcode() == WebAssembly::EXTRACT_EXCEPTION_I32) {
243 if (MI.getOperand(0).isDead())
Heejin Ahnd2a56ac2019-02-26 04:08:49 +0000244 ToDelete.push_back(&MI);
Heejin Ahnd6f48782019-01-30 03:21:57 +0000245 else
246 ExtractInstrs.push_back(&MI);
247 }
Heejin Ahn4934f762018-06-25 01:07:11 +0000248 }
Heejin Ahnd6f48782019-01-30 03:21:57 +0000249 }
Heejin Ahnd2a56ac2019-02-26 04:08:49 +0000250 bool Changed = !ToDelete.empty() || !ExtractInstrs.empty();
251 for (auto *MI : ToDelete)
252 MI->eraseFromParent();
Heejin Ahnd6f48782019-01-30 03:21:57 +0000253 if (ExtractInstrs.empty())
Heejin Ahnd2a56ac2019-02-26 04:08:49 +0000254 return Changed;
Heejin Ahn4934f762018-06-25 01:07:11 +0000255
Heejin Ahnd6f48782019-01-30 03:21:57 +0000256 // Find terminate pads.
257 SmallSet<MachineBasicBlock *, 8> TerminatePads;
258 for (auto &MBB : MF) {
259 for (auto &MI : MBB) {
Heejin Ahn4934f762018-06-25 01:07:11 +0000260 if (MI.isCall()) {
261 const MachineOperand &CalleeOp = MI.getOperand(0);
262 if (CalleeOp.isGlobal() && CalleeOp.getGlobal()->getName() ==
263 WebAssembly::ClangCallTerminateFn)
Heejin Ahnd6f48782019-01-30 03:21:57 +0000264 TerminatePads.insert(getMatchingEHPad(&MI));
Heejin Ahn4934f762018-06-25 01:07:11 +0000265 }
Heejin Ahnd6f48782019-01-30 03:21:57 +0000266 }
267 }
Heejin Ahn4934f762018-06-25 01:07:11 +0000268
Heejin Ahnd6f48782019-01-30 03:21:57 +0000269 for (auto *Extract : ExtractInstrs) {
270 MachineBasicBlock *EHPad = getMatchingEHPad(Extract);
271 assert(EHPad && "No matching EH pad for extract_exception");
Heejin Ahn8b49b6b2019-03-13 00:37:31 +0000272 auto CatchPos = EHPad->begin();
273 if (CatchPos->isEHLabel()) // EH pad starts with an EH label
274 ++CatchPos;
275 MachineInstr *Catch = &*CatchPos;
276
Heejin Ahnd6f48782019-01-30 03:21:57 +0000277 if (Catch->getNextNode() != Extract)
278 EHPad->insert(Catch->getNextNode(), Extract->removeFromParent());
279
280 // - Before:
281 // ehpad:
Heejin Ahn9f96a582019-07-15 22:49:25 +0000282 // %exnref:exnref = catch
Heejin Ahnd6f48782019-01-30 03:21:57 +0000283 // %exn:i32 = extract_exception
284 // ... use exn ...
285 //
286 // - After:
287 // ehpad:
Heejin Ahn9f96a582019-07-15 22:49:25 +0000288 // %exnref:exnref = catch
Heejin Ahnd6f48782019-01-30 03:21:57 +0000289 // br_on_exn %thenbb, $__cpp_exception, %exnref
290 // br %elsebb
291 // elsebb:
292 // rethrow
293 // thenbb:
294 // %exn:i32 = extract_exception
295 // ... use exn ...
Daniel Sanders05c145d2019-08-12 22:40:45 +0000296 Register ExnReg = Catch->getOperand(0).getReg();
Heejin Ahnd6f48782019-01-30 03:21:57 +0000297 auto *ThenMBB = MF.CreateMachineBasicBlock();
298 auto *ElseMBB = MF.CreateMachineBasicBlock();
299 MF.insert(std::next(MachineFunction::iterator(EHPad)), ElseMBB);
300 MF.insert(std::next(MachineFunction::iterator(ElseMBB)), ThenMBB);
301 ThenMBB->splice(ThenMBB->end(), EHPad, Extract, EHPad->end());
302 ThenMBB->transferSuccessors(EHPad);
303 EHPad->addSuccessor(ThenMBB);
304 EHPad->addSuccessor(ElseMBB);
305
306 DebugLoc DL = Extract->getDebugLoc();
307 const char *CPPExnSymbol = MF.createExternalSymbolName("__cpp_exception");
308 BuildMI(EHPad, DL, TII.get(WebAssembly::BR_ON_EXN))
309 .addMBB(ThenMBB)
Sam Cleggef4c66c2019-04-03 00:17:29 +0000310 .addExternalSymbol(CPPExnSymbol)
Heejin Ahnc7397612019-03-05 11:11:34 +0000311 .addReg(ExnReg);
Heejin Ahnd6f48782019-01-30 03:21:57 +0000312 BuildMI(EHPad, DL, TII.get(WebAssembly::BR)).addMBB(ElseMBB);
313
314 // When this is a terminate pad with __clang_call_terminate() call, we don't
315 // rethrow it anymore and call __clang_call_terminate() with a nullptr
316 // argument, which will call std::terminate().
317 //
318 // - Before:
319 // ehpad:
Heejin Ahn9f96a582019-07-15 22:49:25 +0000320 // %exnref:exnref = catch
Heejin Ahnd6f48782019-01-30 03:21:57 +0000321 // %exn:i32 = extract_exception
322 // call @__clang_call_terminate(%exn)
323 // unreachable
324 //
325 // - After:
326 // ehpad:
Heejin Ahn9f96a582019-07-15 22:49:25 +0000327 // %exnref:exnref = catch
Heejin Ahnd6f48782019-01-30 03:21:57 +0000328 // br_on_exn %thenbb, $__cpp_exception, %exnref
329 // br %elsebb
330 // elsebb:
331 // call @__clang_call_terminate(0)
332 // unreachable
333 // thenbb:
334 // %exn:i32 = extract_exception
335 // call @__clang_call_terminate(%exn)
336 // unreachable
337 if (TerminatePads.count(EHPad)) {
338 Function *ClangCallTerminateFn =
339 MF.getFunction().getParent()->getFunction(
340 WebAssembly::ClangCallTerminateFn);
341 assert(ClangCallTerminateFn &&
342 "There is no __clang_call_terminate() function");
Heejin Ahn64517a62019-08-13 17:35:44 +0000343 Register Reg = MRI.createVirtualRegister(&WebAssembly::I32RegClass);
Heejin Ahn831efe02019-08-11 06:24:07 +0000344 BuildMI(ElseMBB, DL, TII.get(WebAssembly::CONST_I32), Reg).addImm(0);
Heejin Ahnd6f48782019-01-30 03:21:57 +0000345 BuildMI(ElseMBB, DL, TII.get(WebAssembly::CALL_VOID))
346 .addGlobalAddress(ClangCallTerminateFn)
Heejin Ahn831efe02019-08-11 06:24:07 +0000347 .addReg(Reg);
Heejin Ahnd6f48782019-01-30 03:21:57 +0000348 BuildMI(ElseMBB, DL, TII.get(WebAssembly::UNREACHABLE));
349
350 } else {
Heejin Ahn66ce4192019-03-16 05:38:57 +0000351 BuildMI(ElseMBB, DL, TII.get(WebAssembly::RETHROW)).addReg(ExnReg);
Heejin Ahnd6f48782019-01-30 03:21:57 +0000352 if (EHInfo->hasEHPadUnwindDest(EHPad))
Heejin Ahn195a62e2019-03-03 22:35:56 +0000353 ElseMBB->addSuccessor(EHInfo->getEHPadUnwindDest(EHPad));
Heejin Ahnd6f48782019-01-30 03:21:57 +0000354 }
Heejin Ahn4934f762018-06-25 01:07:11 +0000355 }
Heejin Ahn4934f762018-06-25 01:07:11 +0000356
Heejin Ahn4934f762018-06-25 01:07:11 +0000357 return true;
358}
Heejin Ahn0bb98652019-01-30 22:44:45 +0000359
360// After the stack is unwound due to a thrown exception, the __stack_pointer
361// global can point to an invalid address. This inserts instructions that
362// restore __stack_pointer global.
363bool WebAssemblyLateEHPrepare::restoreStackPointer(MachineFunction &MF) {
364 const auto *FrameLowering = static_cast<const WebAssemblyFrameLowering *>(
365 MF.getSubtarget().getFrameLowering());
366 if (!FrameLowering->needsPrologForEH(MF))
367 return false;
368 bool Changed = false;
369
370 for (auto &MBB : MF) {
371 if (!MBB.isEHPad())
372 continue;
373 Changed = true;
374
375 // Insert __stack_pointer restoring instructions at the beginning of each EH
376 // pad, after the catch instruction. Here it is safe to assume that SP32
377 // holds the latest value of __stack_pointer, because the only exception for
378 // this case is when a function uses the red zone, but that only happens
379 // with leaf functions, and we don't restore __stack_pointer in leaf
380 // functions anyway.
381 auto InsertPos = MBB.begin();
Heejin Ahn8b49b6b2019-03-13 00:37:31 +0000382 if (InsertPos->isEHLabel()) // EH pad starts with an EH label
383 ++InsertPos;
384 if (InsertPos->getOpcode() == WebAssembly::CATCH)
385 ++InsertPos;
Heejin Ahn0bb98652019-01-30 22:44:45 +0000386 FrameLowering->writeSPToGlobal(WebAssembly::SP32, MF, MBB, InsertPos,
387 MBB.begin()->getDebugLoc());
388 }
389 return Changed;
390}